[nodebb-plugin-desktop-notifications] Desktop Notifications (like GMail)

[cani]

That fixed most of the issues. Chrome asked me to confirm adding the ability to have desktop notifications for my site. Now when I should be getting a notification popup i get an error in console Uncaught TypeError: Cannot read property 'text' of undefined

[baris]

Do you have the line number @cani? Try running in development mode so you can get the exact line number.

[psychobunny]

do you know what notification specifically that was? in any case run npm update, I just added a catch to make sure that we don't show notifications that didn't have any text (for whatever reason)

[cani]

@baris it's line 14 of nodebb-plugin-desktop-notifications/lib/main.js

@psychobunny just a reply to a thread with the text @cani asdf

[psychobunny]

try npm updateand lemme know if it works (version 0.0.6)

push notifications in mobile works only on firefox mobile, haven't tested chrome mobile.

[cani]

I must be doing something wrong, same error just like 24 now
if (data.text) {

[psychobunny]

oh I get it. you need to update nodebb to this commit at minimum:

fix for minification of plugins static directories · NodeBB/NodeBB@5db27a8

Node.js based forum software built for the modern web - fix for minification of plugins static directories · NodeBB/NodeBB@5db27a8

GitHub (github.com)

here's what's blocking you:

also send notification data along with event:new_notification socket … · NodeBB/NodeBB@dc4aeca

Node.js based forum software built for the modern web - also send notification data along with event:new_notification socket … · NodeBB/NodeBB@dc4aeca

GitHub (github.com)

that's why data is undefined

[cani]

That did the trick, Thank you!

[psychobunny]

np, I should have realized. you might have noticed its been activated on the community forum as well

[trevor]

i see.

[quahfamili]

@psychobunny I wanna ask something on mobile. Does it work with safari on iOS? also, i just tried on chrome does not seems to work.

[julian]

@quahfamili you may have to clear your browser cache to see it

[trevor]

var logo = $('.forum-logo').attr('src');

I don't want to use the forum logo, can I use a custom image for this?

[psychobunny]

maybe in the future I could add a way to upload a new picture. for now you could just take that line and replace it with any valid url ex.

var logo = 'http://domain.com/image.png';

[trevor]

Ooo yeah that works.

[psychobunny]

damn desktop notifications! haha. this is going to get annoying eventually

[trevor]

Hah, perhaps in the future.

[Steve]

thanks! will install tonight

[bentael]

[Chris]

This is great. Has anyone else noticed they get double notifications? I do, at least on these boards.

Good work thanks very much for this.