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.

Sign up for GitHub

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

Jump to bottom

Quicker loading of messages with mentions #2366

Closed
elatif2020 opened this issue Nov 29, 2023 · 0 comments · Fixed by #2396
Closed

Quicker loading of messages with mentions #2366

elatif2020 opened this issue Nov 29, 2023 · 0 comments · Fixed by #2396
Assignees
@MeyerPV
Labels
Priority - A V [dev] verified on dev enviroment V [production] verified on production enviroment

Comments

@elatif2020
Copy link
Collaborator

Currently all messages appears together, but previously it was evident the messages with mentions take the longest time to load.
This is probably due to time it takes to get the updated user name from firebase.
We would like to make it faster (and so make all messages appear faster) by immediately displaying the mention with the available userName (as it is stored in the rich text), and updating it afterward if a new - different info is being received from the db.
@MeyerPV MeyerPV mentioned this issue Dec 14, 2023
Merged
@elatif2020 elatif2020 linked a pull request Dec 15, 2023 that will close this issue
Quicker loading of messages with mentions #2366 #2396
Merged
andreymikhadyuk added a commit that referenced this issue Dec 19, 2023
@andreymikhadyuk
Merge pull request #2396 from daostack/CW-2366-quick-loading-mentions
bf03def 
Quicker loading of messages with mentions #2366
@NoamQA NoamQA added the V [dev] verified on dev enviroment label Dec 25, 2023
@NoamQA NoamQA added the V [production] verified on production enviroment label Jan 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MeyerPV MeyerPV

Labels
Priority - A V [dev] verified on dev enviroment V [production] verified on production enviroment
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Quicker loading of messages with mentions #2366 daostack/common-web
3 participants
@MeyerPV @elatif2020 @NoamQA