Skip to content

Feat/notification center profile settings#322

Open
exronnin wants to merge 2 commits into
geevapp:mainfrom
exronnin:feat/notification-center-profile-settings
Open

Feat/notification center profile settings#322
exronnin wants to merge 2 commits into
geevapp:mainfrom
exronnin:feat/notification-center-profile-settings

Conversation

@exronnin
Copy link
Copy Markdown
Contributor

@exronnin exronnin commented Jun 2, 2026

Closes #298
Closes #299


Description

Adds richer user profile/settings management and upgrades the notifications center.

Changes

  • Added editable profile fields for display name, bio, avatar, and privacy preferences.
  • Added account-level notification/privacy controls in settings.
  • Persisted profile preference fields through the user API and Prisma schema migration.
  • Improved notifications page with pagination, refresh, loading/empty states, and deep-link navigation.
  • Synced unread notification counts between the notifications page and navbar bell.
  • Updated notification API responses to include unread counts after fetch/read actions.
  • Added/updated API and notification tests.

Verification

  • Notification tests pass with mock DB: 25 passed.
  • Lint passes with only existing console warnings.
  • Full tsc --noEmit still has pre-existing test typing failures unrelated to these commits.

Checklist

  • I have tested my changes locally
  • I have updated documentation as needed
  • I have run npx prisma generate after schema changes
  • I have run npx prisma migrate dev or npx prisma migrate deploy as appropriate

Post-Merge Steps for Maintainers

If this PR includes changes to the Prisma schema:

  1. Run the following command to apply the migration to your database:

    npx prisma migrate deploy

    or, for local development:

    npx prisma migrate dev
  2. Ensure your CI pipeline runs the migration before tests (add this step if missing):

    - name: Run Prisma Migrate
      run: npx prisma migrate deploy
  3. Make sure the database user in CI has permission to run migrations.


If you have any questions, please comment on this PR.

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Jun 2, 2026

@exronnin Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

[Frontend] Improve notifications center [Frontend] Add profile editing and privacy settings

1 participant