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

https://notices.bulles.fr/api/v3/matching-contexts takes time to get #359

Open
felix-lambert opened this issue Apr 23, 2021 · 2 comments
Open
Assignees
Labels
bug 🐞 Size M 👕 Medium effort/time

Comments

@felix-lambert
Copy link
Collaborator

I can wait sometimes 5 seconds before I can get the result. Sometimes it can be quite fast but it seems to be a bit random (maybe the first loading can take some time, then it's being fast?).

I think we need to investigate because the bubble can take time to appear on mobile so we can have the feeling that it does'nt work

@felix-lambert felix-lambert added bug 🐞 Size M 👕 Medium effort/time labels Apr 23, 2021
@JalilArfaoui
Copy link
Member

I see a 100 ms response time … but the payload is nearly 1 megabyte … What’s the speed of your internet connexion @felix-lambert ? …

In any case, you should not fetch https://notices.bulles.fr/api/v3/matching-contexts on every tab navigation, but rather in the background periodically, so that the list is always ready

@JalilArfaoui JalilArfaoui self-assigned this Apr 23, 2021
@felix-lambert
Copy link
Collaborator Author

felix-lambert commented Apr 26, 2021

Hello Jalil I can confirm that it's not internet connexion related because the chrome browser is rendering very fast. I do see the fetching request being blocked for a few seconds sometimes (not every time) but I will apply your recommendation to make this issue less urgent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Size M 👕 Medium effort/time
Projects
None yet
Development

No branches or pull requests

2 participants