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 bug of DisableGravatar default value #22296

Merged
merged 3 commits into from
Jan 1, 2023

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
backport #22296

Co-authored-by: KN4CK3R <admin@oldschoolhack.me>
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