Skip to content

Conversation

lunny
Copy link
Member

@lunny lunny commented Jan 4, 2023

Unfortunately, #22295 introduced a bug that when set a cached system setting, it will not affect.
This PR make sure to remove the cache key when updating a system setting.

Fix #22332

@lunny lunny added type/bug outdated/backport/v1.18 This PR should be backported to Gitea 1.18 skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. labels Jan 4, 2023
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jan 4, 2023
@yardenshoham yardenshoham added this to the 1.19.0 milestone Jan 4, 2023
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jan 7, 2023
@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 Jan 8, 2023
@lunny lunny merged commit dfd2db5 into go-gitea:main Jan 8, 2023
@lunny lunny deleted the lunny/fix_set_system_settting branch January 8, 2023 13:22
lunny added a commit to lunny/gitea that referenced this pull request Jan 8, 2023
Unfortunately, go-gitea#22295 introduced a bug that when set a cached system
setting, it will not affect.
This PR make sure to remove the cache key when updating a system
setting.

Fix go-gitea#22332
@lunny lunny added the backport/done All backports for this PR have been created label Jan 8, 2023
lunny added a commit that referenced this pull request Jan 9, 2023
zjjhot added a commit to zjjhot/gitea that referenced this pull request Jan 9, 2023
* upstream/main:
  Replace `can not` with `cannot` (go-gitea#22372)
  Fix set system setting failure once it cached (go-gitea#22333)
  Bump json5 from 1.0.1 to 1.0.2 (go-gitea#22365)
  Always reuse transaction (go-gitea#22362)
  make /{username}.png redirect to user/org avatar (go-gitea#22356)
  Remove old HookEventType (go-gitea#22358)
@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
backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. outdated/backport/v1.18 This PR should be backported to Gitea 1.18 skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Toggling Picture and Avatar Configuration switches have no effect
5 participants