Ecomode-auto-addon-todo

[masterwishx]

Signed-off-by: DaRK AnGeL 28630321+masterwishx@users.noreply.github.com

To continue work for #2947 :

To make automatic ECO Mode variable or command experimental.ecomode.start.auto and experimental.ecomode.stop.auto
that run Bypass On then ECO Mode Enable and back instead of users are run two separate commands or variables :

bypass.start + experimental.ecomode.enable
bypass.stop + experimental.ecomode.disable

or

 Enable bypass mode:
upsrw -u admin -p pass -s input.bypass.switch.on=on eaton

Enable ECO mode:
upsrw -u admin -p pass -s input.eco.switchable=enable eaton

Disable bypass mode:
upsrw -u admin -p pass -s input.bypass.switch.off=off eaton

Disable ECO mode:
upsrw -u admin -p pass -s input.eco.switchable=normal eaton

[jimklimov]

@masterwishx : note - this PR got auto-merged with #2947, apparently it was same or subset of that. Was this intended, or did you plan some more work separately from that change?

[masterwishx]

@masterwishx : note - this PR got auto-merged with #2947, apparently it was same or subset of that. Was this intended, or did you plan some more work separately from that change?

Ohh sorry, yep we need more tests and some work maybe for ecomode auto.

I just made new pr from old one #2947 where I changed back to NULL for cmd and deleted all stuff for ecomode auto for continue to work here :(

[jimklimov]

I see. Well, I guess now a new PR is in order to deliver logically named/typed commands. :)

[masterwishx]

I see. Well, I guess now a new PR is in order to deliver logically named/typed commands. :)

Do you mean again to make pr for clean ecomode auto and move it to second new pr?

[jimklimov]

Well... whatever commits were out there, tested by you and @ZivertX in #2947, they were merged and this PR got closed too: 2947 included this PR's chain of commits plus 6cfd10d.

So any other work (if planned) should get done as another PR :)

[masterwishx]

Well... whatever commits were out there, tested by you and @ZivertX in #2947, they were merged and this PR got closed too: 2947 included this PR's chain of commits plus 6cfd10d.

So any other work (if planned) should get done as another PR :)

Yep, already made #2956 :)