Skip to content

fix: Properly annotate settings controller#5416

Merged
elzody merged 5 commits intomainfrom
fix/settings-annotatin
Feb 19, 2026
Merged

fix: Properly annotate settings controller#5416
elzody merged 5 commits intomainfrom
fix/settings-annotatin

Conversation

@juliusknorr
Copy link
Member

  • fix: only let admins request admin settings
  • feat: add behat tests
  • fix: add after-scenario cleanup
  • Resolves: #
  • Target version: main

Summary

TODO

  • ...

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Documentation (manuals or wiki) has been updated or is not required

@juliusknorr juliusknorr requested a review from elzody as a code owner February 19, 2026 19:47
@juliusknorr juliusknorr force-pushed the fix/settings-annotatin branch from 1aac22c to 8d5dd0c Compare February 19, 2026 19:49
@juliusknorr
Copy link
Member Author

/backport! to stable33

@juliusknorr
Copy link
Member Author

/backport! to stable32

@juliusknorr
Copy link
Member Author

/backport! to stable31

@juliusknorr
Copy link
Member Author

/backport! to stable32

Signed-off-by: Elizabeth Danzberger <elizabeth@elzody.dev>
Signed-off-by: Elizabeth Danzberger <elizabeth@elzody.dev>
Signed-off-by: Elizabeth Danzberger <elizabeth@elzody.dev>
Signed-off-by: Elizabeth Danzberger <elizabeth@elzody.dev>
Signed-off-by: Elizabeth Danzberger <elizabeth@elzody.dev>
@elzody elzody force-pushed the fix/settings-annotatin branch from 1527e36 to 4c01acf Compare February 19, 2026 22:13
Copy link
Collaborator

@elzody elzody left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needed to adjust the workflow a bit as the Behat tests were not even running. Also needed to fix a failure once they did run by add index.php to the base URI for richdocuments requests in admin-settings.feature. I will back-port those fixes specifically to the stable32, as that back-port has already been merged.

@elzody elzody merged commit 8dca166 into main Feb 19, 2026
79 checks passed
@elzody elzody deleted the fix/settings-annotatin branch February 19, 2026 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants