-
-
Notifications
You must be signed in to change notification settings - Fork 621
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: deduplicate event handlers #16278
Conversation
Das ist klar. Wir haben aber eigentlich keinen Grund, die auch im Code am CP zu handeln. Stell dir das einfach als das catch-all vor nachdem kein passender CP gefunden wurde. |
Ja, verstehe ich, dennoch sind das eigentlich CP-Aktionen. Dann sollten wir das im CS zumindest entsprechend kommentieren, dass hier nur die Abkürzung genommen wird, da nicht weiter im CP verarbeitet. |
Stell dir das einfach vor wie die anderen Handler auch- nur das hier vorher keine CP-spezifischen Aufrufe mehr getätigt werden. So als würde es keinen handlenden CP geben. |
This PR moves all cp event handlers to cs where logic is globally handled