-
Notifications
You must be signed in to change notification settings - Fork 29
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
Long loading time #31
Comments
Thanks for raising this. It really is a problem and it's definitely an issue on the server side, I'm looking into ways to speed up query lookup times. It looks like those client errors are due to a separate Flagmoji chrome extension. It's failing to convert the location flag emojis on the dashboard into true flag emojis for Windows. I'm not sure why that would be, but it wouldn't be affecting load performance. |
Now it doesn't work at all anymore :/ It loads for a good few minutes before it just stops and displays a "server error" This is the output using Google Chrome:
On Firefox it looks like this:
|
The server is only small and running out of memory on your request. If you try again in a few minutes and wait until it loads, avoid clicking refresh. Does it eventually get through? |
Nope, it still doesn't go through due to a server error |
I'll update the frontend to stop this from happening, but it will be a few days from now. Thanks for letting me know |
Great, thanks tom :) |
Should be working much better now, the data will load in chunks. You may need to clear browser cache with |
@Snailedlt Thanks, that's a good suggestion. The loading is essentially just waiting for another chunk of logged requests to be fetched from the server. It isn't loading anything in particular like certain metrics or processing a particular set of data, that all happens pretty fast. So there isn't really a 'title' that could be displayed as such that would explain to the user what is currently being loaded, it's just waiting for another chunk of requests. It loads requests from most recent to the least recent, so after a while the additional requests may not be relevant for the timescale you are currently looking at (e.g. the last month), so I could hide the loading icon at that point to inform the user that the current page (i.e. currently selected timescale) they are looking at is now 'complete'. If the user selects a larger time period straight afterwards, the loading icon would reappear again. Does that sound like it would be useful, or do you have any other ideas? |
I see what you mean! I agree, I've always thought skeletons would be much cleaner. There's no reason why the majority of the dashboard cannot be rendered with titles before the initial data load. I'll try and get that included soon. |
Awesome! I'll probably have more suggestions down the line, but for now I'm very satisfied with the app except for the things I mentioned already 😄 |
Actually. |
There's a limit to how many requests we can store and process per user while still able to keep the service free. I've had to find a balance between decent server infra while avoiding charging users. I'm working on some efficiency improvements that means we can handle more data per user. A self-hosting solution for the backend should be ready within the next 2 weeks, if you wish you take over the processing and storage. In future, I should be able to offer a paid tier that can handle far more data. |
That makes sense. A paid tier would be great, as well as being great for your motivation ;) |
@tom-draper any updates on this issue? Seems like it still persists |
My analytics site has incredibly long loading time (over 40 seconds)!
https://www.apianalytics.dev/dashboard/fb1f330ea13b4f4c91a8de83895e56e3
That's the analytics for this website: https://markdown-videos-api.jorgenkh.no/
The source code is available here in case you wanna look at it as a part of the debugging: https://github.com/Snailedlt/Markdown-Videos
Here's the console log before the loading is done:
And after the loading is done (the same error continues below the screenshot:
Here's the file it's pointing at:
Seems like this is an error, but perhaps adding some form of caching could also speed things up?
The text was updated successfully, but these errors were encountered: