nodebb-plugin-desktop-notifications icon indicating copy to clipboard operation
nodebb-plugin-desktop-notifications copied to clipboard

Published 20 hours ago • verified publisher icon psychobunny

Request to adapt the plugin to the new versions of nodebb

Open A22632 opened this issue 2 years ago • 5 comments

Request to adapt the plugin to the new versions of nodebb. Thanks.

A22632 avatar Aug 26 '22 06:08 A22632

Hi @A22632 — is there something non-functional in this plugin? I checked it out and all seems ok. There are no admin pages to update and I think the functionality works fine.

@psychobunny will just need to update nbbpm.compatibility to point append || ^2.0.0

julianlam avatar Aug 26 '22 14:08 julianlam

@A22632 I've installed the plugin and is not working. Nor is producing any error.

installed from bash: npm install nodebb-plugin-desktop-notifications activated and rebuild.

Nothing happens

k1r0s avatar Dec 17 '23 12:12 k1r0s

plugin is not compatible with modern versions, tested with 3.5.5

Uncaught (in promise) TypeError: app.alert is not a function

k1r0s avatar Dec 17 '23 13:12 k1r0s

Recommend you use ntfy plugin instead. Any reason you want this one?

julianlam avatar Dec 17 '23 14:12 julianlam

I believe that specific error is covered in the v2 migration guide.

julianlam avatar Dec 17 '23 14:12 julianlam