Skip to content

feat(navbar) notification frontend #93#94

Merged
dizzpy merged 5 commits into
dizzpy:mainfrom
Methexx:main
May 12, 2025
Merged

feat(navbar) notification frontend #93#94
dizzpy merged 5 commits into
dizzpy:mainfrom
Methexx:main

Conversation

@Methexx
Copy link
Copy Markdown
Collaborator

@Methexx Methexx commented May 11, 2025

No description provided.

@Methexx Methexx requested review from Copilot and dizzpy May 11, 2025 00:34
@Methexx Methexx self-assigned this May 11, 2025
@Methexx Methexx added enhancement New feature or request feat add new feature labels May 11, 2025
@vercel
Copy link
Copy Markdown

vercel Bot commented May 11, 2025

@Methexx is attempting to deploy a commit to the dizzpy's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a notifications UI in the NavBar while also updating a navigation route on the ProfilePage.

  • Adds state, refs, and functions to support notifications in the NavBar
  • Implements a notification dropdown with options to mark notifications as read
  • Updates the "Create Your First Post" button on the ProfilePage to navigate to the feed

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/components/Shared/NavBar.jsx Adds notifications state, UI elements, and functions in NavBar
src/[features]/auth/pages/ProfilePage.jsx Updates navigation route for the post creation button
Comments suppressed due to low confidence (2)

src/components/Shared/NavBar.jsx:131

  • The 'Bell' component is used but not imported, which will result in a runtime error. Please add an import statement for 'Bell' from 'lucide-react'.
<Bell size={24} />

src/[features]/auth/pages/ProfilePage.jsx:580

  • [nitpick] The button label 'Create Your First Post' may not match the new route '/feed'. Please verify if the label or route needs to be updated to ensure a consistent user experience.
onClick={() => navigate('/feed')}

@dizzpy dizzpy merged commit 3811ddc into dizzpy:main May 12, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request feat add new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants