Mattzobricks / MattzoControllers

MattzoController firmware
20 stars 10 forks source link

MTC4BT: Unify alternative direction label in config between channels and actions #8

Closed raymondbrink closed 1 year ago

raymondbrink commented 2 years ago

Channels now expect a value of reverse for the alternate direction. Actions require backward. Both should use the same value.

Mattzobricks commented 1 year ago

It might be a good idea to remain backwards compatible when implementing this change, so user configuration will not fail after the firmware upgrade.

Mattzobricks commented 1 year ago

To Dos:

raymondbrink commented 1 year ago

The keyword will be "backward", so a channel definition can be:

{
    "channel": "A",
    "attachedDevice": "motor",
    "direction": "backward" // "reverse" is still supported for backward compatibility
}

Event triggers based on the "dirchanged" event will remain the same as before:

{
    "source": "loco",
    "eventType": "dirchanged",
    "value": "backward"
}
Mattzobricks commented 1 year ago

Merged. Closing issue.