nextcloud / nextcloud/spreed

chatBlocks follow-ups

Open
#15,668 1 comment 0 reactions 1 assignee View on GitHub

@Antreesy is already working on this.

Since Aug 14, 2025.

bug feature: chat 💬 feature: frontend 🖌️
Dominant language
JavaScript
Stars
2.2k
Forks
586
Avg merge
18h 27m
Merged PRs (30d)
333

Description

Follow-ups to https://github.com/nextcloud/spreed/pull/10084:

Link

  • 🚧 lastMessage can be made a separate chatBlock, if not present in initial context
  • 🐛 'Loading new messages ...' is shown too often (id mismatch?)
  • 🐛 In sidebar, there is 'No messages' shown before loader

TODO:

  • 🐛 threadBlocks are required to properly keep chat context: https://github.com/nextcloud/spreed/pull/15616
  • 🚧 unifying onRouteChange and handleStartGettingMessagesPreconditions or extracting common logic later.
    At the moment:
    • handleStartGettingMessagesPreconditions checks if user 'canGetMessages';
    • onRouteChange triggers only afterwards under separate conditions, assuming the check above is true
  • 👀 contextMessageId can still be 0 as per previous unchanged logic. Check if valid
  • 👀 compact view doesn't have lastMessage to rely on
    • Compact view changed to still poll with lastMessage
  • 👀 federated conversations aren't tested and missing lastMessageId

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.