Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changing permissions for federated shares #11022

Closed
kobergj opened this issue Feb 3, 2025 · 4 comments · Fixed by #11041
Closed

Changing permissions for federated shares #11022

kobergj opened this issue Feb 3, 2025 · 4 comments · Fixed by #11041
Assignees

Comments

@kobergj
Copy link
Collaborator

kobergj commented Feb 3, 2025

PATCHing federated shares is not supported by the server. We should block editing federated shares in the webfrontend. See comment - proposal got rejected

@kobergj
Copy link
Collaborator Author

kobergj commented Feb 3, 2025

Proposal Rejected!

We need to implement patching ocm shares. We can use a frontend only approach: Delete the old share and create a new one.

@kobergj kobergj changed the title Block changing permissions for federated shares Changing permissions for federated shares Feb 3, 2025
@kobergj kobergj moved this from Qualification to Backlog in Infinite Scale Team Board Feb 5, 2025
@kobergj
Copy link
Collaborator Author

kobergj commented Feb 17, 2025

Current state: We'll try to implement this in the backend because it is less effort.

@kobergj
Copy link
Collaborator Author

kobergj commented Feb 24, 2025

@2403905 I tried with current master. Changing permissions works (web enables the buttons correctly). But uploading a file fails. Removing and Readding the share works properly. Could you please recheck?

@kobergj kobergj reopened this Feb 24, 2025
@github-project-automation github-project-automation bot moved this from Done to In progress in Infinite Scale Team Board Feb 24, 2025
@2403905
Copy link
Contributor

2403905 commented Feb 25, 2025

@kobergj Thank you for your notice. Firxed in a reva cs3org/reva#5096

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants