Skip to content

Frontend: Invalidate ping query to update message badges#8754

Merged
nabramow merged 2 commits into
developfrom
na/web/message-read-notifs
May 23, 2026
Merged

Frontend: Invalidate ping query to update message badges#8754
nabramow merged 2 commits into
developfrom
na/web/message-read-notifs

Conversation

@nabramow

Copy link
Copy Markdown
Member

Frontend pingQueryKey invalidation: makes the UI update immediately. Without it, the frontend won't refetch the ping query until its natural 10-second polling interval fires — so the bell badge would still show the old stale count for up to 10 seconds, even though the backend data is already correct.

Closes #8312

Testing

Explain how you tested this PR and give clear steps so the reviewer can replicate.

Web frontend checklist

  • There are no console warnings when running the app
  • Added tests where relevant
  • Clicked around my changes running locally and it works
  • Checked Desktop, Mobile and Tablet screen sizes

For maintainers

  • Maintainers can push commits to my branch
  • Maintainers can merge this PR for me

@vercel

vercel Bot commented May 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
couchers Ready Ready Preview May 23, 2026 8:44am

Request Review

@nabramow
nabramow marked this pull request as ready for review May 22, 2026 22:53
@nabramow
nabramow merged commit c93053d into develop May 23, 2026
6 checks passed
@nabramow
nabramow deleted the na/web/message-read-notifs branch May 23, 2026 12:26
@github-actions github-actions Bot added the release notes: pending Add to stuff that should be included in release notes label May 23, 2026
@github-actions

Copy link
Copy Markdown
Contributor

📝 Release Notes

This PR should be included in the release notes.

Suggested release note:

Fixed message badges so read messages are marked as read immediately instead of appearing unread for several seconds
🤖 Bot Debug Information

Model: couchers.openai.gpt-5.4
Decision: include
Reasoning: This fixes a noticeable user-facing bug where message/notification badges could continue showing unread items for up to 10 seconds after messages were read. It improves the accuracy and responsiveness of message status in the UI, which users would notice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release notes: pending Add to stuff that should be included in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Read messages still unread

2 participants