Skip to content

Make API Endpoint to set the ReadOnly status of a Wiki#1009

Merged
outdooracorn merged 6 commits into
mainfrom
T410395-readonlywiki
Nov 26, 2025
Merged

Make API Endpoint to set the ReadOnly status of a Wiki#1009
outdooracorn merged 6 commits into
mainfrom
T410395-readonlywiki

Conversation

@rosalieper
Copy link
Copy Markdown
Contributor

Bug: T410395

@rosalieper rosalieper changed the title [WIP]: Make API Endpoint to set the ReadOnly status of a Wiki Make API Endpoint to set the ReadOnly status of a Wiki Nov 24, 2025
Comment thread app/Http/Controllers/Backend/WikiReadOnlyController.php Outdated
Comment thread app/Http/Controllers/Backend/WikiReadOnlyController.php
Comment thread app/Http/Controllers/Backend/WikiReadOnlyController.php Outdated
Copy link
Copy Markdown
Contributor

@tarrow tarrow left a comment

Choose a reason for hiding this comment

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

just approving the last cosmetic changes from @outdooracorn

@outdooracorn outdooracorn merged commit 8857ef6 into main Nov 26, 2025
5 checks passed
@outdooracorn outdooracorn deleted the T410395-readonlywiki branch November 26, 2025 11:43
rosalieper added a commit that referenced this pull request Nov 27, 2025
Follow up refactoring for #1009. I
believe it is important for tests to reflect exactly how we expect
things to work. As we updated the value of `readOnly` on the task
description, I thought it would be good to update the test too.

Bug: T410395
deer-wmde pushed a commit that referenced this pull request Dec 15, 2025
deer-wmde pushed a commit that referenced this pull request Dec 15, 2025
Follow up refactoring for #1009. I
believe it is important for tests to reflect exactly how we expect
things to work. As we updated the value of `readOnly` on the task
description, I thought it would be good to update the test too.

Bug: T410395
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.

3 participants