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 dashboard ignored system setting cache (#21621) #21759

Merged

Conversation

lunny
Copy link
Member

@lunny lunny commented Nov 10, 2022

backport #21621

This is a performance regression from #18058

Signed-off-by: Andrew Thornton [email protected]
Co-authored-by: Andrew Thornton [email protected]

This is a performance regression from go-gitea#18058

Signed-off-by: Andrew Thornton <[email protected]>
Co-authored-by: Andrew Thornton <[email protected]>
@lunny lunny added performance/speed performance issues with slow downs skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. labels Nov 10, 2022
@lunny lunny added this to the 1.18.0 milestone Nov 10, 2022
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Nov 10, 2022
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Nov 10, 2022
@lunny lunny merged commit b9dcf99 into go-gitea:release/v1.18 Nov 10, 2022
@lunny lunny deleted the lunny/fix_performance_dashboard2 branch November 10, 2022 11:41
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. performance/speed performance issues with slow downs skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants