Skip to content

feat: add AppContext refresh methods and notifications mark-as-read#284

Merged
3m1n3nc3 merged 1 commit intogeevapp:mainfrom
yusuftomilola:feat/appcontext-refresh-notifications-281-282
Apr 24, 2026
Merged

feat: add AppContext refresh methods and notifications mark-as-read#284
3m1n3nc3 merged 1 commit intogeevapp:mainfrom
yusuftomilola:feat/appcontext-refresh-notifications-281-282

Conversation

@yusuftomilola
Copy link
Copy Markdown

Summary

  • Adds refreshPosts() and refreshPostDetail(postId) to AppContextType contract and implements both in AppProvider — fetches from /api/posts and /api/posts/:id respectively and reconciles local state against server responses
  • Updates the notifications page to add per-notification "Mark as read" buttons and a bulk "Mark all as read" action that call the existing PATCH /api/notifications/:id endpoint and update unread count in local state without a full reload
  • Adds a typed Notification interface to the page and a 'use client' directive for Next.js client component compatibility

closes #281
closes #282

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Apr 23, 2026

@yusuftomilola 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

@3m1n3nc3 3m1n3nc3 merged commit b50cfb9 into geevapp:main Apr 24, 2026
2 checks 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.

Complete notifications API integration including mark-as-read Finalize AppContext API refresh contract and remove local-only mutations

2 participants