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: new templates #15293

Draft
wants to merge 20 commits into
base: master
Choose a base branch
from
Draft

OCPP: new templates #15293

wants to merge 20 commits into from

Conversation

premultiply
Copy link
Member

@premultiply premultiply commented Aug 7, 2024

  • 🔌 add protocol field to templates to distinguish different implementations for the same product (e.g. localapi & ocpp)
  • 🌟 presets in defaults.yaml now can have requirements and protocol

TODOs

  • verify evcc configure works correctly
  • config ui: generalize naming adjustment @naltatis
  • decide multiple-protocol support @andig

@premultiply premultiply added the devices Specific device support label Aug 7, 2024
@naltatis
Copy link
Member

naltatis commented Aug 9, 2024

Wo wir hier gerade beim Namensaufräumen sind. Wie unterscheiden sich denn die beiden Victron-Implementierungen?

Bildschirmfoto 2024-08-09 um 21 05 41

Ist das eine die "Charging Station NS" und wenn ja welche? Oder sind das auch zwei unterschiedliche Implementierungen für das gleiche Gerät?

Bildschirmfoto 2024-08-09 um 21 07 13

\cc @philipptrenz @andig

@naltatis
Copy link
Member

Das Victron Thema hat sich erledigt: 260c13e

@premultiply premultiply mentioned this pull request Aug 12, 2024
1 task
@andig
Copy link
Member

andig commented Aug 12, 2024

Mir ist nicht so richtig klar, was wir hier erreichen wollen. Insbesondere: warum sollten wir zwei Implementierungen für eine Box anbieten? Auch bei diesem PR wäre es schön, ihn in kleinere Portionen zu zerteilen. Z.b. sind requirements von presets auch alleine nützlich.

@premultiply
Copy link
Member Author

premultiply commented Aug 13, 2024

Hier kommen eine Ladung neue OCPP-Templates rein.
Gleichtzeitig die erforderlichen Umbauten um dies in UI, configure und Doku entsprechend schön aufzubereiten - auch wenn es stellenweise mehrere Wege gibt eine Box anzusprechen.

@andig
Copy link
Member

andig commented Aug 13, 2024

auch wenn es stellenweise mehrere Wege gibt eine Box anzusprechen.

Das sollten wir nach Möglichkeit nicht tun. Lasst uns gerne diskutieren wo das absolut notwendig ist.

@premultiply
Copy link
Member Author

premultiply commented Aug 13, 2024

Das wäre wenn überhaupt nur ein Sonderfall für Boxen die über mehrere Wege angesprochen werden können und dabei verschiedene Features haben (z.B. Keba und Sungrow). Steht hier nicht im Fokus.

Hier geht vor allem mal darum einheitlich anzeigen, suchen, sortieren und filtern zu können welche Box mit welcher Schnittstelle angesprochen/konfiguriert werden muss.

@philipptrenz
Copy link
Contributor

Wo wir hier gerade beim Namensaufräumen sind. Wie unterscheiden sich denn die beiden Victron-Implementierungen?

Sorry für die späte Rückmeldung, bin gerade im Urlaub. Aber ja, so hätte ich es auch gelöst 👍🏼
Der Hinweis auf das notwendige Sponsorship muss noch entfernt werden, richtig?

@github-actions github-actions bot added the stale Outdated and ready to close label Aug 28, 2024
@github-actions github-actions bot closed this Sep 2, 2024
@premultiply premultiply reopened this Sep 2, 2024
@premultiply premultiply added backlog Things to do later enhancement New feature or request and removed stale Outdated and ready to close labels Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Things to do later devices Specific device support enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants