Skip to content
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.

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

[Bug] Retention policy breaks sorting by activity by prune chat messages #2673

Closed
ankar84 opened this issue Nov 26, 2020 · 3 comments
Closed

Comments

@ankar84
Copy link
Contributor

ankar84 commented Nov 26, 2020

Description:

We implemented Rocket Chat Retention Policy in our deployment and now have an issue with sorting by activity on mobile client, web version works good.
As I get - prune messages process updates chats timestamps and it gets upper in Chat List sorted by activity. But in fact, there is no new messages in that chats and they shouldn't appear on top in sort by activity.
I think web version Sidebar works different with chats activity timestamps (more correct, IMHO)

Environment Information:

  • Rocket.Chat Server Version: 3.7.1
  • Rocket.Chat App Version: 4.12.1
  • Device Name: Xiaomi Mi 9 SE
  • OS Version: Android 10

Steps to reproduce:

  1. Enable Retention Policy to prune messages in all chats once a day
  2. Set sorting by activity
  3. Web version sorting is correct, but mobile version sorting broken.

Expected behavior:

Sort by activity on Mobile should be same as on Web

Actual behavior:

Additional context:

No

@ankar84
Copy link
Contributor Author

ankar84 commented Dec 14, 2020

That that appears only in mobile version and not in Web or Desktop version, so as a workaround now I hide that old chats with Swipe left - Hide.

@diegolmello
Copy link
Member

Hey, @ankar84 .
I'm not sure it's a fix, but a new patch with #2763 just landed Google Play (iOS to be approved by Apple).
Can you test it?
Thanks!

@ankar84
Copy link
Contributor Author

ankar84 commented Jan 6, 2021

Hey, @ankar84 .
I'm not sure it's a fix, but a new patch with #2763 just landed Google Play (iOS to be approved by Apple).
Can you test it?
Thanks!

Yeah, @diegolmello
Looks like #2763 solved that issue and chat list in RCRN experimental application version 4.13.1.22186 without empty chats from far past.
Thanks for your great job, Diego!

@ankar84 ankar84 closed this as completed Jan 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants