Skip to content

fix: delete space action showing for space managers#2649

Merged
JammingBen merged 3 commits into
mainfrom
fix/space-manager-space-delete
Jun 17, 2026
Merged

fix: delete space action showing for space managers#2649
JammingBen merged 3 commits into
mainfrom
fix/space-manager-space-delete

Conversation

@JammingBen

Copy link
Copy Markdown
Member

When a space manager, that is not a (space) admin, disables a space, the UI was previously showing the delete action. While the API should prevent that, the UI should also not show the action.

Fixes this issue by reloading the space permissions after disabling or enabling a space.

However, there are still questions that need clarification in the future:

refs opencloud-eu/opencloud#1799

@JammingBen JammingBen self-assigned this Jun 9, 2026
@JammingBen JammingBen requested review from AlexAndBear and kulmann June 9, 2026 07:32
When a space manager, that is not a (space) admin, disables a space,
the UI was previously showing the delete action. While the API should
prevent that, the UI should also not show the action.

Fixes this issue by reloading the space permissions after disabling or
enabling a space.
Comment thread packages/web-app-files/src/composables/actions/spaces/useSpaceActionsRestore.ts Outdated
Comment thread packages/web-app-files/src/composables/actions/spaces/useSpaceActionsRestore.ts Outdated
@JammingBen JammingBen force-pushed the fix/space-manager-space-delete branch from d494d15 to 147ef8e Compare June 17, 2026 08:58
@JammingBen JammingBen requested a review from kulmann June 17, 2026 09:12
@JammingBen JammingBen merged commit 450ddd7 into main Jun 17, 2026
30 checks passed
@JammingBen JammingBen deleted the fix/space-manager-space-delete branch June 17, 2026 11:03
@openclouders openclouders mentioned this pull request Jun 17, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants