-
Notifications
You must be signed in to change notification settings - Fork 196
Conversation
In origin-discovery... | ||
|
||
- Run `node src/listener/listener.js --continue-file=continue --webhook=http://localhost:3456/events` | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@micahalcorn I have at least one undefined variable
I think it is the DATABASE_URL
.
- Is this the correct format
postgresql://localhost/notification
? - Is there another variable I need?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, I forgot one. Thanks for testing. 😁
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Natasha08 try now with 37dd006 🤞
@micahalcorn that seemed to work! So now on the DApp side, I'm seeing this in the console when I try to enable notifications. I don't see an error as a user, but I don't see notifications also |
@Natasha08 have you taken some action that would trigger a notification? For example, subscribe to notifications from a seller account in browser A and then create an offer from a buyer account in browser B? |
@micahalcorn Yes I have been doing the example you mentioned. The origin notifications widget shows the notification, but after getting the browser window asking for permission to show notifications, I do not see browser notifications |
@micahalcorn I finally have notifications working with these additions.
And
|
For reference 📖