Skip to content

Notifications Hub #195

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.

Sign up for GitHub

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

Jump to bottom
Open
cindy-x-liang wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Notifications Hub #195

cindy-x-liang wants to merge 15 commits into from

Conversation

@cindy-x-liang
Copy link
Contributor

@cindy-x-liang cindy-x-liang commented Feb 23, 2025 •
edited
Loading

Overview

Implemented the frontend for notifications hub

Changes Made

Favorite Item Notif

  • A class that ideally stores the favorite item notif data from a /getAllNotifs endpoint from backend

Implemented table view for notifs hub

  • NotificationViewCell contains the customized table view cell that shows the relevant notif data, a button to open the favorite items view controller, and a red icon on the favorite icon to show if a notification has been clikced into or not
  • currently if you click on the button and return to the notif hub the red dot will disappear

Next Steps (optional)

Implement networking

Screenshots (optional)

image image

MrPeterss
MrPeterss requested changes Mar 19, 2025
View reviewed changes
Copy link
Contributor

@MrPeterss MrPeterss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm getting a lot of errors and it seems like an entire file is missing. not sure what happened but I will try to see if I can find the other file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MrPeterss MrPeterss MrPeterss requested changes

@JaysonHahn JaysonHahn Awaiting requested review from JaysonHahn

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cindy-x-liang @MrPeterss