Skip to content

Feature: Notification Panel with Categorized Activity Feed T113#119

Closed
7JankiPanchal wants to merge 5 commits intogdg-charusat:mainfrom
7JankiPanchal:chore/upgrade-vite
Closed

Feature: Notification Panel with Categorized Activity Feed T113#119
7JankiPanchal wants to merge 5 commits intogdg-charusat:mainfrom
7JankiPanchal:chore/upgrade-vite

Conversation

@7JankiPanchal
Copy link
Copy Markdown
Contributor

@7JankiPanchal 7JankiPanchal commented Feb 27, 2026

Team Number : Team 113

Related Issue

Closes #106

Description:

Implemented a Notification Panel that provides a categorized activity feed for users, including:

Duel invites – notifications when another user invites you to a duel.

Achievements – updates on milestones, like problem-solving streaks.

System messages – maintenance alerts or announcements.

Unread count badge – shows number of unread notifications.

Mark as read / mark all as read – users can manage notification status.

Fully integrated with mock authentication for local testing.

Type of Change

New Feature (non-breaking)

Bug fix

Breaking Change

Checklist

Notification types handled: duel, achievement, system

Unread count updated dynamically

Notifications persist using localStorage

UI matches existing theme and design system

Screenshot (128) Screenshot (129) Screenshot (130)

@krishnapaljadeja krishnapaljadeja self-requested a review February 27, 2026 17:03
@krishnapaljadeja krishnapaljadeja added the invalid-pr PR is missing required information label Feb 27, 2026
@krishnapaljadeja
Copy link
Copy Markdown
Contributor

⚠️ PR Validation Failed

Hey @7JankiPanchal! Your PR is missing a required field:

Team Number missing — add your team number anywhere in the PR description.
Example: Team 07

How to fix:

  1. Click the pencil ✏️ icon on your PR description
  2. Add your team number (e.g. Team 07)
  3. Save — this check will re-run automatically

GDG CHARUSAT Open Source Contri Sprintathon

@krishnapaljadeja krishnapaljadeja added invalid-pr PR is missing required information needs-review Valid issue-linked PR awaiting review and removed invalid-pr PR is missing required information labels Feb 27, 2026
@krishnapaljadeja
Copy link
Copy Markdown
Contributor

✅ PR Validation Passed

Hey @7JankiPanchal! Your PR looks good. Here is what we found:

Field Value
Team Number Team 113
Linked Issue Closes #106

A maintainer will review your PR within 24–48 hours. Stay responsive to feedback!

GDG CHARUSAT Open Source Contri Sprintathon

@krishnapaljadeja krishnapaljadeja added needs-review Valid issue-linked PR awaiting review and removed needs-review Valid issue-linked PR awaiting review labels Feb 28, 2026
@aaleya5
Copy link
Copy Markdown
Contributor

aaleya5 commented Feb 28, 2026

hey @7JankiPanchal
please take a pull from main and then commit.
there are multiple errors in your code.

@krishnapaljadeja krishnapaljadeja added needs-review Valid issue-linked PR awaiting review and removed needs-review Valid issue-linked PR awaiting review labels Mar 1, 2026
@aaleya5
Copy link
Copy Markdown
Contributor

aaleya5 commented Mar 1, 2026

image solve these conflicts. @7JankiPanchal

@krishnapaljadeja krishnapaljadeja added needs-review Valid issue-linked PR awaiting review and removed needs-review Valid issue-linked PR awaiting review labels Mar 1, 2026
@aaleya5
Copy link
Copy Markdown
Contributor

aaleya5 commented Mar 1, 2026

hey @7JankiPanchal there are numerous syntactical errors likely caused by the merge you did
kindly run project review the errors and then commit.

@aaleya5 aaleya5 closed this Mar 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review Valid issue-linked PR awaiting review pr-noted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature: Notification Panel with Categorized Activity Feed T113

3 participants