Skip to content

fix: Update permission checks for archiving forms and transfer ownership#3028

Merged
Chartman123 merged 1 commit into
mainfrom
fix/permissionChecks
Nov 21, 2025
Merged

fix: Update permission checks for archiving forms and transfer ownership#3028
Chartman123 merged 1 commit into
mainfrom
fix/permissionChecks

Conversation

@Chartman123
Copy link
Copy Markdown
Collaborator

This fixes #3024 by adjusting the permission checks in the backend. It also adds two fixes to disable settings in the front end if the user is not the owner.

Signed-off-by: Christian Hartmann chris-hartmann@gmx.de

Signed-off-by: Christian Hartmann <chris-hartmann@gmx.de>
@Chartman123 Chartman123 added the bug Something isn't working label Nov 5, 2025
@Chartman123 Chartman123 self-assigned this Nov 5, 2025
@Chartman123 Chartman123 requested a review from susnux November 5, 2025 23:05
@Chartman123 Chartman123 added 3. to review Waiting for reviews regression Regression of a previous working feature labels Nov 5, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 5, 2025

Codecov Report

❌ Patch coverage is 15.38462% with 11 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
lib/Controller/ApiController.php 15.38% 11 Missing ⚠️

📢 Thoughts on this report? Let us know!

@Chartman123
Copy link
Copy Markdown
Collaborator Author

/backport to stable5.2

@Chartman123 Chartman123 merged commit a0ebec1 into main Nov 21, 2025
52 of 54 checks passed
@Chartman123 Chartman123 deleted the fix/permissionChecks branch November 21, 2025 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug Something isn't working regression Regression of a previous working feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Form cannot be closed manually

2 participants