[Bug]: Talk system messages delay loading
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 2.2k
- Forks
- 586
- Avg merge
- 18h 27m
- Merged PRs (30d)
- 333
Description
⚠️ This issue respects the following points: ⚠️
- This is a bug, not a question or a configuration/webserver/proxy issue.
- This issue is not already reported on Github OR Nextcloud Community Forum (I've searched it).
- Nextcloud Server is up to date. See Maintenance and Release Schedule for supported versions.
- I agree to follow Nextcloud's Code of Conduct.
Bug description
One of our company-wide chatrooms recently got cleaned up, i.e. ~50 old/inactive/deleted users were removed at once. This generated the according amount of system messages mentioning the removal of each specific user. These messages also get merged into a single appearing line just fine as expected. Though due to them being so many, I assume it takes quite a while for the client to start loading in previous messages. So when a user opens up the chat, he sees the system messages only (basically an empty chat), having to wait for previous messages to load in.
Steps to reproduce
- Create a group with 50+ users.
- Remove 50 users from the chat at once.
- Reload the chat window (Ctrl + F5)
Expected behavior
The merged system messages (single line) and X previous messages load in right away. I.e. system messages do not count against the default count of messages to be loaded when opening a chat.
Nextcloud Server version
32
Operating system
None
PHP engine version
None
Web server
None
Database engine version
None
Is this bug present after an update or on a fresh install?
None
Are you using the Nextcloud Server Encryption module?
None
What user-backends are you using?
- Default user-backend (database)
- LDAP/ Active Directory
- SSO - SAML
- Other
Configuration report
List of activated Apps
Nextcloud Signing status
Nextcloud Logs
Additional info
No response
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.
Research direction
No files or tests are named. Start by tracing the chat history loading and pagination path used when a chat window reloads, then inspect how merged system messages affect the initial message count; done means previous messages appear immediately alongside the merged system messages after reproducing the 50-user removal case.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100