In-App notifications.

[hackrush01]

Inclusion of an in-app notification system would be helpful in many cases such as if we want the people to move from slack to discord, we could just send an in-app notification for the same. It could be extended to other functionalities such as informing about incoming tips, new content, new upload from subscriptions etc.

[tzarebczan]

Notification scenarios, including the ones mentioned above:

• Publish completed
• Download completed
• Incoming tip
• Incoming content payment
• Received LBC
• New content in subbed channels
• Trending content
• Pushed notifications from LBRY about app features / news / content / creators
• LBRY app updates
• System outages
• Vanity URL outbid

[alyssaoc]

@seanyesmunt This should be considered for an epic

[btzr-io]

WIP (Just the ui, no redux logic yet)

@seanyesmunt any ideas or feedback on this?

[kauffj]

Ooooooh I love this. Not Sean but some quick comments from me:

1. It'd be cool if it showed the number of notifications
2. I'd keep the design consistent. Keep the spacing allocated for the image/icon the same, as well as the header formatting (unless you just couldn't get this to work...)
3. Consider adding elements that allow marking notifications as read or to hide them (see how FB manages theirs)
4. Less stylistic weight could be given to the header

[btzr-io]

Some minor updates, still working on 3

[btzr-io]

[neb-b]

This is awesome @btzr-io.

I agree with @kauffj's comments. Maybe add an x in the corner when you hover over one notification to remove it from the stack. For the redux side this should live inlbry-redux. Probably in the notifications reducer.

I think to keep it simple we can just start with adding newly subscribed content and your publish confirmations: "Your file at lbry://something is published" or something similar.

[neb-b]

Also this should use theFileTile component to keep it consistent with search/related content.

[btzr-io]

publish confirmations: "Your file at lbry://something is published"

@seanyesmunt There is a modal for that, should I remove it?

[btzr-io]

[neb-b]

@btzr-io Sorry I missed your comment.

Maybe we should show the desktop notification if the app is closed, but not if it's open? Will this contain an "in-app" notification when something happens? It would be the same text as the text inside the drop down, but shown right when something happens? Maybe that should be it's own issue.

[tzarebczan]

Another use case now is comment notifications.

[kauffj]

@seanyesmunt https://github.com/lbryio/internal-apis/pull/1326 will make 6 existing notifications consumable via API and make it much easier to add new ones whenever we want

Expect work to begin on this next week if we can find the time (and we probably should)

[kauffj]

@sean ensure it works as intended

[kauffj]

• Fix spacing
• Mark individual notification read
• Unify colors with mobile