Skip to content

Conversation

lunny
Copy link
Member

@lunny lunny commented Dec 31, 2022

#18058 made a mistake. The disableGravatar's default value depends on OfflineMode. If it's true, then disableGravatar is true, otherwise it's false. But not opposite.

@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 Dec 31, 2022
@lunny lunny added this to the 1.19.0 milestone Dec 31, 2022
@lunny lunny added the backport/done All backports for this PR have been created label Dec 31, 2022
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Dec 31, 2022
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (main@dce8887). Click here to learn what that means.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main   #22296   +/-   ##
=======================================
  Coverage        ?   48.15%           
=======================================
  Files           ?     1043           
  Lines           ?   142404           
  Branches        ?        0           
=======================================
  Hits            ?    68575           
  Misses          ?    65645           
  Partials        ?     8184           
Impacted Files Coverage Δ
models/system/setting.go 57.47% <100.00%> (ø)
modules/setting/picture.go 78.18% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@KN4CK3R
Copy link
Member

KN4CK3R commented Jan 1, 2023

We should check/enforce the positive form of settings. So EnableXXX instead of DisableXXX. Makes it easier to work with.

@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 1, 2023
@lunny
Copy link
Member Author

lunny commented Jan 1, 2023

make L-G-T-M work

@lunny lunny merged commit 9c8fc7f into go-gitea:main Jan 1, 2023
@lunny lunny deleted the lunny/fix_bug_system_setting branch January 1, 2023 12:19
lunny added a commit that referenced this pull request Jan 1, 2023
zjjhot added a commit to zjjhot/gitea that referenced this pull request Jan 3, 2023
* upstream/main:
  Add deprecated warning for DISABLE_GRAVATAR and ENABLE_FEDERATED_AVATAR (go-gitea#22318)
  Unify hashing for avatar (go-gitea#22289)
  fix: code search title translation (go-gitea#22285)
  Update Gmail mailer configuration (go-gitea#22291)
  Fix due date rendering the wrong date in issue (go-gitea#22302)
  Fix get system setting bug when enabled redis cache (go-gitea#22295)
  Restructure `webhook` module (go-gitea#22256)
  Reminder for no more logs to console (go-gitea#22282)
  Fix bug of DisableGravatar default value (go-gitea#22296)
  Upgrade go-chi to v5.0.8 (go-gitea#22304)
  [skip ci] Updated licenses and gitignores
  Use ErrInvalidArgument in packages (go-gitea#22268)
  Changelog v1.18.0 (go-gitea#22215) (go-gitea#22269)
  Support estimated count with multiple schemas (go-gitea#22276)
  Add Gentoo to the from package providers (go-gitea#22284)
  Fix sitemap (go-gitea#22272)
  Add `sync_on_commit` option for push mirrors api (go-gitea#22271)
  Fix key signature error page (go-gitea#22229)
@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.

5 participants