[BUG] Edit Grid Meter Node: "L3 Current (A)" missing, L2 Current double instead

victronenergy / node-red-contrib-victron

MIT License

87 stars 18 forks source link

[BUG] Edit Grid Meter Node: "L3 Current (A)" missing, L2 Current double instead #166

Closed GallusMax closed 1 year ago

GallusMax commented 1 year ago

The Node Edit form leaves out the "L3 Current (A)" item. - see Screenshot.

venusGridMeterNode

Steps to reproduce the behavior:

Using node "Grid Meter"
Click on Selector "Measurement"
See the list of possible items corresponding to topics
"L2 Current (A)" appears twice, but "L3 Current (A)" is missing

Software (please complete the following information):

node-red-contrib-victron version: v1.4.29
Venus OS version: 3.0~20

Please forgive the late report if fixed with v1.4.30 ;-)

dirkjanfaber commented 1 year ago

Thanks for reporting. Got the report yesterday evening too and the fix is here: https://github.com/victronenergy/node-red-contrib-victron/compare/master...faberd/grid-l3. The second L2 is in fact showing L3, but just has a typo in the description.

GallusMax commented 1 year ago

Whow, that is really fast! I just filed a pull request fixing the typo, which is obsolete now. (Please take a look on the other pull request regarding /Ac/Power sum value.)