Skip to content

fix(chat): correctly update last message and unread counter from polling#16892

Merged
Antreesy merged 1 commit intomainfrom
fix/noid/last-message-update
Jan 28, 2026
Merged

fix(chat): correctly update last message and unread counter from polling#16892
Antreesy merged 1 commit intomainfrom
fix/noid/last-message-update

Conversation

@Antreesy
Copy link
Copy Markdown
Contributor

β˜‘οΈ Resolves

  • Fix incorrect updates in LeftSidebar
    • lastMessage is used later to updated LeftSidebar with subline and unread counter
    • lastMessage should only be of a list of rendered messages for that list
    • this doesn't include hidden system messages like reactions

πŸ–ŒοΈ UI Checklist

πŸ–ΌοΈ Screenshots / Screencasts

🏚️ Before

2026-01-28_17h56_05.mp4

🏑 After

2026-01-28_17h54_40.mp4

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Integrations with Files sidebar and other apps
    • Not risky to browser differences / client

- lastMessage is used later to updated LeftSidebar with subline and unread counter
- lastMessage should only be of a list of rendered messages for that list
- this doesn't include hidden system messages like reactions

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy merged commit 114fe6c into main Jan 28, 2026
57 checks passed
@Antreesy Antreesy deleted the fix/noid/last-message-update branch January 28, 2026 22:15
@Antreesy
Copy link
Copy Markdown
Contributor Author

/backport to stable33

@Antreesy
Copy link
Copy Markdown
Contributor Author

/backport to stable32

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

Labels

feature: chat πŸ’¬ Chat and system messages feature: frontend πŸ–ŒοΈ "Web UI" client

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants