Skip to content

Notification UI#63

Merged
portableDD merged 3 commits intoNexacore-Org:mainfrom
benedictfred:feat/notification
Aug 22, 2025
Merged

Notification UI#63
portableDD merged 3 commits intoNexacore-Org:mainfrom
benedictfred:feat/notification

Conversation

@benedictfred
Copy link
Copy Markdown
Contributor

📌 Notification UI

📋 Description

I built the notification dropdown according the figma design provided. I also set it up for data fetching using the backend API_URL.

🛠 Changes Implemented

  • I created a file NotificationDropdown.tsx and wired it up with the bell icon on the Navbar.
  • I created api/notificationService.ts file and added getNotification, markAsRead and markAllAsRead utility functions.
  • I created the useNotifications and useMarkAllAsRead hooks.

✅ How to Test

  1. Setup: Clone the feat/notification branch.
  2. Run Tests: Run npm install and npm run dev to start the local dev environment.
  3. Expected Output: Navigate to /dashboard and click on the bell icon to see the notifcation dropdown.

🔍 Screenshots (if applicable)

Desktop Screen
Screencast from 2025-08-22 18-44-54.webm

Mobile Screen
Screencast from 2025-08-22 18-47-00.webm

🚀 Checklist

  • Code follows project style guidelines.
  • Changes are tested and verified.
  • Documentation is updated (if needed).

🔗 Related Issues

Additional Information

Currently the dropdown renders a dummy data as notifications. This is because the getNotifications functions requires a userId which is gotten when the user signs in. Apart from that, the utility functions are available and ready to use. All one has to do is just to pass the userId into the hooks and loop over the data variable returned by tanstack query.

@portableDD
Please review. Thanks 🙏.

Copy link
Copy Markdown
Contributor

@portableDD portableDD left a comment

Choose a reason for hiding this comment

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

Good Work, Fix conflict and we are good to go

@benedictfred
Copy link
Copy Markdown
Contributor Author

@portableDD
I have fixed the conflicts.

@portableDD portableDD merged commit 5d915e8 into Nexacore-Org:main Aug 22, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Issue: Implement Notifications UI and Data Fetching

2 participants