-
Notifications
You must be signed in to change notification settings - Fork 81
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add notifications feed #5434
Labels
Milestone
Comments
This exists which is the first cut at this: couchers/app/proto/notifications.proto Line 17 in 29163a6
|
@pepsicolacat Tagging you here to finish off the pop up over the pop up issue. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We want to make it easier for users to see when an action related to them happens. There is a backend api for this you can connect to:
rpc ListNotifications(ListNotificationsReq) returns (ListNotificationsRes);
Acceptance Criteria
ListNotifications
to the frontendNOTE: Backend might not have everything we need, that's okay for now. Just do what we have, if you notice something missing make a note what and we can create a new ticket (or feel free to make the ticket).
DESIGN
Desktop - friend request example
Desktop - notification settings hover/click example
Mobile example
The text was updated successfully, but these errors were encountered: