Add user notification data service (fix #563)

Sage-Bionetworks / sagebio-collaboration-portal

Collaboration Portal developed by Sage Bionetworks

1 stars 0 forks source link

Add user notification data service (fix #563) #565

Closed tschaffter closed 4 years ago

tschaffter commented 4 years ago

This PR fixes #563.

Add user notification data service
Show badge on user notification icon when num notification > 0
Refactor PageTitle service and correctly show the number of notifications to review

tschaffter commented 4 years ago

phccp_num_notification

ychae commented 4 years ago

Thanks for the fix @tschaffter! I tested both as admin and test user and it's working perfectly for the test user! I highly doubt that anyone will really be using the admin account extensibly so I think this is all good to merge in. I've included a gif below of the test:

notifications

tschaffter commented 4 years ago

Thanks @ychae !