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

Load messages and conversation in background when push message is received #4379

Open
2 tasks
mahibi opened this issue Oct 24, 2024 · 0 comments
Open
2 tasks
Labels
1. to develop Accepted and waiting to be taken care of (should be only set by nextcloud employees) enhancement New feature or request feature: 📵 offline support

Comments

@mahibi
Copy link
Collaborator

mahibi commented Oct 24, 2024

when a push message is received

  • the offline messages should be updated immediately
  • the offline conversations should be updated immediately
    in background.

So when entering the chat, there is no need to fetch messages from server but the initial loading from server can be skipped. This will improve user experience when entering a chat as it feels like everything is already in place and doesn't have to be loaded.
Also, this will be useful to have data offline as soon as possible in case the connection is lost.

This will require to move some logic from OfflineFirstChatRepository to ViewModel or a DomainModel so the Repository can also be used from NotificationWorker.

  • discuss if fetching messages and conversations for every push message is causing too much load for the server?
  • is fetching conversation list necessary? Clicking notification directs to the chat and skips the conversation list
@mahibi mahibi added 1. to develop Accepted and waiting to be taken care of (should be only set by nextcloud employees) enhancement New feature or request feature: 📵 offline support labels Oct 24, 2024
@mahibi mahibi moved this to 🆕 New to evalutate in 💬 Talk team Oct 24, 2024
@rapterjet2004 rapterjet2004 self-assigned this Nov 14, 2024
@rapterjet2004 rapterjet2004 removed their assignment Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of (should be only set by nextcloud employees) enhancement New feature or request feature: 📵 offline support
Projects
Status: 🆕 New to evalutate
Development

No branches or pull requests

2 participants