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

Only show last 6 version updates on admin settings screen #3368

Closed
wants to merge 1 commit into from

Conversation

DieselTech
Copy link
Collaborator

Changed

  • Changed: Show only the last 6 updates in the settings#admin-system screen. This stops the page from trying to load 20+ versions going back well over 2 years.

Honestly even 6 feels like a lot. We could lower it since going back that far brings us to pretty old versions that aren't relevant anymore.

Simple split to only show the last 6 update log results. This stops the page from trying to load 20+ versions going back well over 2 years.

Honestly even 6 feels like a lot. We could lower it if we wanted.
@majora2007
Copy link
Member

I'd like this implementation to change. Have the API take a int (6) so that the UI can request only the lastest releases. Then refactor all the way down to the VersionService code.

This way we don't do extra work for that one flow, but we can keep it existing for others. Use a default value of 0 to ignore.

@majora2007
Copy link
Member

I took care of this in my feature/random branch.

@majora2007 majora2007 closed this Dec 22, 2024
@majora2007 majora2007 deleted the feature/paginate-update-history branch December 22, 2024 15:50
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.

2 participants