chatBlocks follow-ups
Open
@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:handleStartGettingMessagesPreconditionschecks if user 'canGetMessages';onRouteChangetriggers 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
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.