Skip to content

fix: resolve customer unread indicators#62

Merged
mafirs merged 1 commit into
labring:mainfrom
mafirs:fix/user-unread
Jun 4, 2026
Merged

fix: resolve customer unread indicators#62
mafirs merged 1 commit into
labring:mainfrom
mafirs:fix/user-unread

Conversation

@mafirs

@mafirs mafirs commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix customer unread indicators that could remain stuck when the latest chat message is not visible to customers.

Root Cause

Customer ticket detail only loads non-internal messages, but the customer ticket list and read/unread filtering used the latest message across all chat messages. If the latest message was internal, customers could not see or mark that message as read, so the sidebar unread dot could reappear after switching tickets.

Changes

  • Filter customer ticket-list latest-message lookup to customer-visible messages.
  • Apply the same customer-visible filtering to read/unread ticket filtering.
  • Make read-status sending wait for an open WebSocket connection.
  • Only clear local unread state after read-status sending succeeds.

Verification

  • Ran bun run typecheck
  • Ran git diff --check upstream/main..origin/fix/user-unread
  • Confirmed PR branch comparison against labring/tentix:main is ahead by 1 commit with 4 changed files.

Notes

Please merge with Create a merge commit.

@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown

🤖 Fork Repository Check Complete

✅ Basic security scan completed
✅ Dependency check completed
✅ Linting check completed
✅ Build test completed

This PR from a fork repository has passed basic safety checks. A maintainer will review the changes before running full CI/CD pipeline.

This is an automated safety check for contributions from fork repositories.

@mafirs mafirs merged commit 424f7b1 into labring:main Jun 4, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant