Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: Notification if update available #87

Closed
Scrounger opened this issue Dec 15, 2022 · 6 comments
Closed

Feature: Notification if update available #87

Scrounger opened this issue Dec 15, 2022 · 6 comments
Labels
enhancement New feature or request in progress Working on it

Comments

@Scrounger
Copy link
Contributor

Describe the solution you'd like
It would be great if the adapter would also check for updates and send a notification.
Think that actually fits great to this adapter, what do you think?

@Scrounger Scrounger added the enhancement New feature or request label Dec 15, 2022
@ciddi89
Copy link
Collaborator

ciddi89 commented Dec 15, 2022

I think it's a good Idea! Thanks for it!
Please try the GitHub Version. You will find the new option under notifications.

Screenshot 2022-12-15 at 19 05 44

IMG_0597

Scrounger added a commit to Scrounger/ioBroker.device-watcher that referenced this issue Dec 17, 2022
* optionally show adapter name in notification

* added mqtt nuki and mqtt Z2M

* Update main.js

added comment

* improved show adaptername in notifies iobroker-community-adapters#83

* corrected spelling error iobroker-community-adapters#86

* fixed devices are not reported in case battery is 0% iobroker-community-adapters#86

* fixed that adaptername was also shown at devicename on all lists

* add point at work in progress

* fixed last commit

* added mqtt nuki

* added mqtt nuki hub

* added mqtt client zigbee2mqtt iobroker-community-adapters#82

* Added Synology Chat to notification service iobroker-community-adapters#85

* updated readme

* added apc ups adapter

* changed connection state for apcups

* using uptime for unifi

* unifi improvements

* Feature: Notification if update available iobroker-community-adapters#87

Co-authored-by: Christian Behrends <mail@christian-behrends.de>
@Scrounger
Copy link
Contributor Author

@ciddi89

oh great, that was fast. Thank you!
I will try it in the next days and give a feedback

@Scrounger
Copy link
Contributor Author

@ciddi89

It works that it detects when updates for new adapters are available.
That was not quite what I meant, sorry. I had not thought of this option. A good example why feature request should be well formulated ;-)

My idea is actually to get an info when there is e.g. a firmware update for a device.
Let's take for example a unifi device. Here there is the datapoint unifi.0.default.devices.X.upgradable which tells that there is a new firmware.

The arrApart.js would then need a property upgrade for example:

	unifi: {
		Selektor: 'unifi.*.devices.*.uptime',
		adapterID: 'unifi',
		adapter: 'Unifi',
		battery: 'none',
		reach: '.state',
		isLowBat: 'none',
		id: 'none',
		upgrade: '.upgradable'
	},

What do you think?

@ciddi89
Copy link
Collaborator

ciddi89 commented Dec 17, 2022

@Scrounger ah okay yes this makes sense. I wasn't sure about it, because only a few adapter gives this information. But never mind, I think it makes sense and we can add it.

@ciddi89 ciddi89 added the in progress Working on it label Dec 17, 2022
@ciddi89
Copy link
Collaborator

ciddi89 commented Dec 18, 2022

@Scrounger Okay I think it is now as you imagined it.
Screenshot 2022-12-18 at 16 06 22
IMG_0621

@Scrounger
Copy link
Contributor Author

Great thanks for implementing this! Works fine.

ciddi89 added a commit that referenced this issue Dec 19, 2022
-   (Scrounger) optionally show adapter name in notification
-   (ciddi89) optionally receive a message when an update for an device is available [#87](#87)
-   (Scrounger) update message for shelly adapter added
-   (ciddi89) update message for unifi devices added
-   (ciddi89) fixed devices are not reported in case battery is 0% [#86](#86)
-   (ciddi89) SynoChat added [#85](#85)
-   (ciddi89) MQTT NukiHub, MQTT-Clien Zigbee2MQTT added ([#82](#82))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request in progress Working on it
Projects
None yet
Development

No branches or pull requests

2 participants