dupondje / nrgkick-ha

Home Assistant - NRGKick Integration
14 stars 3 forks source link

feat: changed native_step of charge_current_limit due to nrgkick app … #14

Closed fano0001 closed 2 months ago

fano0001 commented 4 months ago

Hi, i found a toggle in the NRGKick APP today, to toggle the steps to set the current charge limit from int to float.

So it is possible via the NRGKick APP to set the ampere value to e.g. 14.5

dupondje commented 4 months ago

But, if you don't have the setting enabled, does setting a float value via HA work? Otherwise we need to check the value of the setting first.

fano0001 commented 4 months ago

Will check it tomorrow, but i think its only a setting in the NRGKick app

fano0001 commented 4 months ago

My test was successful. My test steps was the following.

  1. disable the "decimal" feature in the NRGKick app.
  2. Set the chage limit to 13.5 ampere

when i go to the NRGKick app the ui shows 13 ampere, but when i enable the feature in the NRGKick app the ui shows 13.5. So the feature Switch is only to toggle the APP ui from int to float, the NRGKick itself uses the value from HA.

PHOTO-2024-02-26-19-59-08 PHOTO-2024-02-26-19-59-46

PHOTO-2024-02-26-19-59-46

fano0001 commented 2 months ago

Any problems with the PR?

dupondje commented 2 months ago

I forgot honestly. Sorry for that!