Skip to content

Conversation

@com6056
Copy link
Contributor

@com6056 com6056 commented Dec 14, 2025

Trying to use the Home Assistant plugin instead of OCPP for my Wallbox Pulsar Plus (OCPP seems to be very finicky with these).

When I manually pause the charging in the app or via Home Assistant, it gets stuck though since it doesn't recognize paused as a valid status:

[lp-1  ] ERROR 2025/12/14 13:33:10 charger status: unknown charge status: Paused
image

@evcc-bot evcc-bot added devices Specific device support go Pull requests that update go code labels Dec 14, 2025
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey there - I've reviewed your changes - here's some feedback:

  • Confirm whether Home Assistant/Wallbox can send Paused (capitalized) or other variants and, if so, normalize the status string (e.g., to lower-case) before lookup so the new paused mapping is reliably applied.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- Confirm whether Home Assistant/Wallbox can send `Paused` (capitalized) or other variants and, if so, normalize the status string (e.g., to lower-case) before lookup so the new `paused` mapping is reliably applied.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@com6056
Copy link
Contributor Author

com6056 commented Dec 14, 2025

Hey there - I've reviewed your changes - here's some feedback:

  • Confirm whether Home Assistant/Wallbox can send Paused (capitalized) or other variants and, if so, normalize the status string (e.g., to lower-case) before lookup so the new paused mapping is reliably applied.

Prompt for AI Agents

Please address the comments from this code review:

## Overall Comments
- Confirm whether Home Assistant/Wallbox can send `Paused` (capitalized) or other variants and, if so, normalize the status string (e.g., to lower-case) before lookup so the new `paused` mapping is reliably applied.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

strings.ToLower is already being used:

if status, ok := chargeStatusMap[strings.ToLower(strings.TrimSpace(state))]; ok {

@andig andig enabled auto-merge (squash) December 15, 2025 08:14
@andig andig merged commit 5bbcee3 into evcc-io:master Dec 15, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devices Specific device support go Pull requests that update go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants