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

fix: fix long initial time with huge print history #1714

Merged
merged 7 commits into from
Dec 31, 2023

Conversation

meteyou
Copy link
Member

@meteyou meteyou commented Dec 25, 2023

Description

This PR fix a issue with long initial times with huge print histories. It only loads the first 100 entries and the rest only when the user clicks on a button.

Related Tickets & Documents

fixes #1468

Mobile & Desktop Screenshots/Recordings

none

[optional] Are there any post-deployment tasks we need to perform?

none

@meteyou meteyou requested review from dw-0 and rackrick December 25, 2023 19:22
Copy link
Contributor

Language file analysis report:

File Missing Keys Unused Keys
en.json 0 0

Copy link
Contributor

Language file analysis report:

File Missing Keys Unused Keys
de.json 0 0
en.json 0 0

src/locales/de.json Outdated Show resolved Hide resolved
src/locales/en.json Outdated Show resolved Hide resolved
Copy link
Contributor

Language file analysis report:

File Missing Keys Unused Keys
de.json 0 0
en.json 0 0
sv.json 412 12

1 similar comment
Copy link
Contributor

Language file analysis report:

File Missing Keys Unused Keys
de.json 0 0
en.json 0 0
sv.json 412 12

@meteyou meteyou merged commit 74ff01b into mainsail-crew:develop Dec 31, 2023
10 checks passed
@meteyou meteyou deleted the fix/loading-time-history branch December 31, 2023 07:27
@jLynx
Copy link

jLynx commented Dec 31, 2023

Thanks for completing this, excited to try it out. Happy 2024!

@jLynx
Copy link

jLynx commented Jan 1, 2024

Issue seems to be mostly solved now. Now it takes no longer than 8 seconds which is a massive improvement over 15-30 seconds previously. Looking at the natwork traffic, 2 seconds is for the css file. So I'm assuming thst gets dynamically generated?

@meteyou
Copy link
Member Author

meteyou commented Jan 1, 2024

@jLynx no... All files are static.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Slow website UI load times
3 participants