Skip to content

Fix user settings sidebar showing disabled features on some pages#36958

Merged
lunny merged 4 commits intogo-gitea:mainfrom
bircni:fix/disabled-ldap-36954
Mar 23, 2026
Merged

Fix user settings sidebar showing disabled features on some pages#36958
lunny merged 4 commits intogo-gitea:mainfrom
bircni:fix/disabled-ldap-36954

Conversation

@bircni
Copy link
Copy Markdown
Member

@bircni bircni commented Mar 23, 2026

Move UserDisabledFeatures context data into a shared SettingsCtxData middleware for the /user/settings route group, so it is set consistently on all pages (including Notifications, Actions, etc.) instead of only on the handlers that remembered to set it individually.

Fixes #36954

Move UserDisabledFeatures context data into a shared SettingsCtxData
middleware for the /user/settings route group, so it is set consistently
on all pages (including Notifications, Actions, etc.) instead of only
on the handlers that remembered to set it individually.

Fixes go-gitea#36954
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Mar 23, 2026
@github-actions github-actions bot added the modifies/go Pull requests that update Go code label Mar 23, 2026
@bircni bircni marked this pull request as ready for review March 23, 2026 15:34
@bircni bircni requested a review from wxiaoguang March 23, 2026 15:47
@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 Mar 23, 2026
@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 Mar 23, 2026
@lunny lunny added type/bug backport/v1.25 This PR should be backported to Gitea 1.25 reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. labels Mar 23, 2026
@lunny lunny enabled auto-merge (squash) March 23, 2026 21:05
@lunny lunny merged commit 86401fd into go-gitea:main Mar 23, 2026
26 checks passed
@GiteaBot GiteaBot added this to the 1.26.0 milestone Mar 23, 2026
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Mar 23, 2026
GiteaBot pushed a commit to GiteaBot/gitea that referenced this pull request Mar 23, 2026
…-gitea#36958)

Move UserDisabledFeatures context data into a shared SettingsCtxData
middleware for the /user/settings route group, so it is set consistently
on all pages (including Notifications, Actions, etc.) instead of only on
the handlers that remembered to set it individually.

Fixes go-gitea#36954
@GiteaBot GiteaBot added the backport/done All backports for this PR have been created label Mar 23, 2026
silverwind pushed a commit that referenced this pull request Mar 24, 2026
…6958) (#36969)

Backport #36958 by @bircni

Move UserDisabledFeatures context data into a shared SettingsCtxData
middleware for the /user/settings route group, so it is set consistently
on all pages (including Notifications, Actions, etc.) instead of only on
the handlers that remembered to set it individually.

Fixes #36954

Co-authored-by: Nicolas <bircni@icloud.com>
zjjhot added a commit to zjjhot/gitea that referenced this pull request Mar 24, 2026
* giteaofficial/main:
  [skip ci] Updated translations via Crowdin
  Make PUBLIC_URL_DETECTION default to "auto" (go-gitea#36955)
  Fix user settings sidebar showing disabled features on some pages (go-gitea#36958)
  Update GitHub Actions to latest major versions (go-gitea#36964)
  Fix various trivial problems (go-gitea#36953)
  Rework checkbox styling, remove `input` border hover effect (go-gitea#36870)
  Add `DEFAULT_DELETE_BRANCH_AFTER_MERGE` setting (go-gitea#36917)
  Update Nix flake (go-gitea#36943)
  Update to eslint 10 (go-gitea#36925)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/done All backports for this PR have been created backport/v1.25 This PR should be backported to Gitea 1.25 lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/go Pull requests that update Go code type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

User settings sidebar shows disabled features on some pages for external/LDAP users

4 participants