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

Feature request: Add powerplay for the charging point #156

Closed
kevinmelchers opened this issue Jan 7, 2025 · 15 comments
Closed

Feature request: Add powerplay for the charging point #156

kevinmelchers opened this issue Jan 7, 2025 · 15 comments
Labels
help wanted Extra attention is needed

Comments

@kevinmelchers
Copy link

kevinmelchers commented Jan 7, 2025

In the Zonneplan App you can see the powerplay compensation for the charging point. It would be nice to add the property to the integration.

Screenshot_Zonneplan_Powerplay

Response in Dutch is possible.

@fsaris
Copy link
Owner

fsaris commented Feb 14, 2025

Kunnen jullie logs aanleveren? Dan kan ik de juiste velden achterhalen en opnemen in de integratie.

Ga naar de integratie https://my.home-assistant.io/redirect/integration/?domain=zonneplan_one
en klik "Debug logging inschakelen"
wacht een paar minuten zodat nieuwe data is opgehaald
schakel logging weer uit door op de zelde knop te klikken

De log wordt automatisch als download aangeboden.

Mail deze evt naar mij franssaris at gmail.com of je kunt hem hier in het ticket toevoegen (maar dan zou ik wel persoonlijke data uit de log filteren).

@fsaris fsaris added the help wanted Extra attention is needed label Feb 14, 2025
fsaris added a commit that referenced this issue Feb 14, 2025
Attempt to make powerplay values visible
fsaris added a commit that referenced this issue Feb 14, 2025
@Arnoudd
Copy link
Contributor

Arnoudd commented Feb 24, 2025

Kunnen jullie logs aanleveren? Dan kan ik de juiste velden achterhalen en opnemen in de integratie.

Ga naar de integratie https://my.home-assistant.io/redirect/integration/?domain=zonneplan_one en klik "Debug logging inschakelen" wacht een paar minuten zodat nieuwe data is opgehaald schakel logging weer uit door op de zelde knop te klikken

De log wordt automatisch als download aangeboden.

Mail deze evt naar mij franssaris at gmail.com of je kunt hem hier in het ticket toevoegen (maar dan zou ik wel persoonlijke data uit de log filteren).

Ik heb een log bestand per mail toegestuurd.

Zou ook mooi zijn als je de desired_distance_in_kilometers en de desired_end_time instelbaar kan maken? :)

@fsaris
Copy link
Owner

fsaris commented Feb 24, 2025

The main versie toont de power play waardes. Worden flex genoemd

@Arnoudd
Copy link
Contributor

Arnoudd commented Feb 24, 2025

The main versie toont de power play waardes. Worden flex genoemd

Not the laadpaal powerplay vergoeding. image

@fsaris
Copy link
Owner

fsaris commented Feb 24, 2025

Weet je zeker dat je de main/beta versie geïnstalleerd hebt?

Daar is Charge point flex result nu beschikbaar (misschien staat de sensor uitgeschakeld)

@Arnoudd
Copy link
Contributor

Arnoudd commented Feb 24, 2025

Weet je zeker dat je de main/beta versie geïnstalleerd hebt?

Daar is Charge point flex result nu beschikbaar (misschien staat de sensor uitgeschakeld)

2024.10.1

I only got the binary_sensor.charge_laadpaal_charge_point_dynamic_charging_flex_enabled

And

binary_sensor.charge_laadpaal_charge_point_dynamic_charging_flex_suppressed
imageimage

@fsaris
Copy link
Owner

fsaris commented Feb 24, 2025

Please try 2025.2.0 Pre-release

@Arnoudd
Copy link
Contributor

Arnoudd commented Feb 24, 2025

Please try 2025.2.0 Pre-release

image

🙏
Thanks!
What was missing?

@fsaris
Copy link
Owner

fsaris commented Feb 24, 2025

Didn't properly tag the new version

@fsaris
Copy link
Owner

fsaris commented Feb 24, 2025

Zou ook mooi zijn als je de desired_distance_in_kilometers en de desired_end_time instelbaar kan maken? :)

Helaas iets wat ik zelf niet direct kan doen omdat ik geen laadpaal heb en de feature dus ook niet in de app. Daardoor kan ik het niet reverse engineeren en/of testen.

@kevinmelchers
Copy link
Author

Zou ook mooi zijn als je de desired_distance_in_kilometers en de desired_end_time instelbaar kan maken? :)

Helaas iets wat ik zelf niet direct kan doen omdat ik geen laadpaal heb en de feature dus ook niet in de app. Daardoor kan ik het niet reverse engineeren en/of testen.

Wij hebben er wel een. Je mag altijd vragen om iets te testen.

@Arnoudd
Copy link
Contributor

Arnoudd commented Feb 24, 2025

Zou ook mooi zijn als je de desired_distance_in_kilometers en de desired_end_time instelbaar kan maken? :)

Helaas iets wat ik zelf niet direct kan doen omdat ik geen laadpaal heb en de feature dus ook niet in de app. Daardoor kan ik het niet reverse engineeren en/of testen.

Dat is dan reverse engineeren met een tool als burp suite? Of iets anders?

Ik heb daar nog geen ervaring mee maar wil best eens een poging wagen. Ik heb wel alleen iOS om mee te testen.

@Arnoudd
Copy link
Contributor

Arnoudd commented Feb 26, 2025

Nog een vraagje. Wanneer de laadsessie beëindigd is toont de Zonneplan app weer een powerplay vergoeding van 0,00 euro. Maar in de integratie gaat de waarde dan naar 'unknown'. Kennelijk wordt er dan geen waarde meer gegeven vanuit de api.

Kan de integratie dan ook 0,00 tonen? Dat maakt het makkelijker om een utility_meter te gebruiken. Of een grafiek. Beide kunnen slecht tegen de 'unknown' values.

Ik heb een pull request gemaakt. Kijk maar of je het ermee eens bent.
#162

Arnoudd added a commit to Arnoudd/home-assistant-zonneplan-one that referenced this issue Feb 26, 2025
Feature request: Add powerplay for the charging point fsaris#156
@Arnoudd
Copy link
Contributor

Arnoudd commented Feb 28, 2025

Met deze toevoeging werkt het heel mooi. Mooi hetzelfde als de app weergeeft.

Tijd voor een kleine release? Daarna kunnen de mensen met laadpaal die info ook posten naar https://onbalansmarkt.com/live namelijk.

Volgens mij moet er nog een release gemaakt worden voordat iedereen die laatste toevoeging meekrijgt toch?

@fsaris
Copy link
Owner

fsaris commented Feb 28, 2025

Done

@fsaris fsaris closed this as completed Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants