Skip to content

fix(TalkDashboard): show unread mentions content #15689

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
Antreesy merged 1 commit into from
Aug 20, 2025
Merged

fix(TalkDashboard): show unread mentions content #15689

Antreesy merged 1 commit into from
Aug 20, 2025

Conversation

@Antreesy
Copy link
Contributor

@Antreesy Antreesy commented Aug 20, 2025

☑️ Resolves

  • Ref discussion from Talk dashboard follow up #15114
    • As mentioned in the issue, skipping lastMessage brings more issues than benefits. A lot of components relies on this data:
      • Dashboard content
      • Message polling logic
      • New messages asterisk
      • Messages processing
    • Server-wide, it doubles request size that is made with 30-sec interval (IMO, modified since should be optimized instead)

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🏚️ Before 🏡 After
2025-08-20_10h29_36 2025-08-20_10h29_57
16.4 kB transferred for 160 rooms 34.9 kB transferred for 160 rooms

🏁 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

@Antreesy Antreesy added this to the 🪺 Next Major (32) milestone Aug 20, 2025
@Antreesy Antreesy requested review from DorraJaouad and ShGKme August 20, 2025 08:42
@Antreesy Antreesy self-assigned this Aug 20, 2025
@Antreesy
Copy link
Contributor Author

Antreesy commented Aug 20, 2025

/backport to stable31

@Antreesy Antreesy merged commit f3435de into main Aug 20, 2025
53 checks passed
@Antreesy Antreesy deleted the fix/15114/always-fetch-last-message branch August 20, 2025 09:28
@backportbot backportbot bot mentioned this pull request Aug 20, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DorraJaouad DorraJaouad DorraJaouad approved these changes

@ShGKme ShGKme Awaiting requested review from ShGKme

Assignees

@Antreesy Antreesy

Labels

3. to review bug feature: frontend 🖌️ "Web UI" client

Projects

None yet

Milestone

v22.0.0-beta.1

Development

Successfully merging this pull request may close these issues.

3 participants

@Antreesy @DorraJaouad