Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ocpp: cache and re-use initial status #16859

Merged
merged 8 commits into from
Oct 26, 2024
Merged

Ocpp: cache and re-use initial status #16859

merged 8 commits into from
Oct 26, 2024

Conversation

andig
Copy link
Member

@andig andig commented Oct 25, 2024

May fix #15677

@andig andig added the enhancement New feature or request label Oct 25, 2024
@andig andig requested a review from premultiply October 25, 2024 13:42
@andig andig force-pushed the fix/ocpp-initial-status branch from 3977372 to 05e588f Compare October 25, 2024 14:58

// apply cached status if available
instance.WithChargepointStatusByID(cp.id, func(status *core.StatusNotificationRequest) {
if _, err := cp.OnStatusNotification(status); err == nil {
Copy link
Member Author

Choose a reason for hiding this comment

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

@premultiply bin noch gespannt, was hier bei >1 Connector passiert. Denn aktuell speichern wir nur einen Status- das müssten dann aber mehrere sein? Das ist aber erstmal ein Luxusproblem...

Copy link

@etecprojekt etecprojekt Oct 28, 2024

Choose a reason for hiding this comment

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

@andig / @premultiply:
OCPP ABL Box leider mit
cannot create charger 'wb_re_2': cannot create charger type 'template': cannot create charger type 'ocpp': timeout
evcc 0.131.2 (520d54b) - sowohl mit der nightly als auch der 0.131.2.
wb_re_2 ist der 2. Ladepunkt der Box.
Wenn ich ein Issue eröffnen soll, bitte kurz melden.

Copy link
Member Author

Choose a reason for hiding this comment

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

Lass uns erstmal nach der Mennekes schauen.

@andig andig merged commit ce14d93 into master Oct 26, 2024
6 checks passed
@andig andig deleted the fix/ocpp-initial-status branch October 26, 2024 10:57
andig added a commit that referenced this pull request Oct 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OCPP: Mennekes Smart Ladestation seit 0.130.8 Fehlerhaft
2 participants