DoorBird plugin feature request - Single device option

[kwijibo007]

As requested by berkinet here: [urlhttps://forums.indigodomo.com/viewtopic.php?p=164567#p164567][/url]

I have a suggestion: In the Device configuration dialog allow the user to select from a set of states to be reported, probably by check boxes. Then, those who want separate devices can have their needs met, and so can users like myself who only want one device.

I can add this functionality without too much effort. How would you like the bell/motion event custom states to read? Would a simple "last motion detected" & "last bell press" detected suffice? This would display a time/date value showing the last time the bell/motion were detected. This way you can create a trigger based on any change to this field?

I was thinking something like...

• Select the activity to report in the Device table state column::
  O System Status
  O Motion events
  O Doorbell keypress

Then based on the selection above, two of the following three options would appear (I.e. the option matching the choice above would be hidden...

• Select additional information to display in custom states
  [ ] System Status
  [ ] Motion events
  [ ] Doorbell keypress

Then the states associated with the current 3 device types (plus one additional state equivalent to the current onOffState) would be included for the options selected plus one additional state equivalent to the current onOffState. I.e.
For System Status

• build_number
  firmware
  mac_address
  model
  deviceLastChange
  deviceOnOffState

For Motion Events

• motionOnOffState
  motionLastChange

For Doorbell keypress Events

• doorbellOnOffState
  doorbellLastChange

In this manner, the user could build three separate devices, each essentially identical to what you have today. Or, they could build a device combining the capabilities of two or three of the current devices. Note, for the "main" function (to be displayed in the Device State column) the onOff and lastChange states would be essentially duplicated. While not necessary, this would allow for consistency in all devices.

[kwijibo007]

Thanks for the suggestion, I think it has real merit.

I'd like to meet you three quarters of the way there

I think each Doorbird needs a "primary" device. This device stores the username/password, accepts and decrypts the UDP packets and stores universal states (firmware / mac/ etc.).

To the primary device I'll add additional states for:

deviceLastChange
deviceOnOffState
motionOnOffState
motionLastChange
doorbellOnOffState
doorbellLastChange

For simplicity, these states will be updated whenever the device status, motion or doorbell are activated.

As you suggested, I'll add an option to choose what the device table state column displays - either device, motion or doorbell.

The additional Motion and Doorbell devices will remain as optional devices (as they currently are).

In this configuration you can have multiple devices (as it currently stands), or one single device displaying all states with an option to choose the functionality of the device table state column (as requested).

How does this sound?

Pete

[berkinet]

Thanks for the suggestion, I think it has real merit.

I'd like to meet you three quarters of the way there

How does this sound?

Pete

It sounds perfect.

[kwijibo007]

Just released an update to the plugin (0.4.0) which includes the changes discussed in this topic: viewtopic.php?f=287&t=21090

Pete