Skip to content

[stable30] fix: for shares allow document creation only for users that ca…#4809

Merged
elzody merged 1 commit into
nextcloud:stable30from
IONOS-Productivity:fix/shares-disable-doc-creation-v30-shares
Jun 4, 2025
Merged

[stable30] fix: for shares allow document creation only for users that ca…#4809
elzody merged 1 commit into
nextcloud:stable30from
IONOS-Productivity:fix/shares-disable-doc-creation-v30-shares

Conversation

@tanyaka
Copy link
Copy Markdown

@tanyaka tanyaka commented May 26, 2025

Summary

This fix assures files creation from templates in public share is not possible for users with read-only permissions.
Users with read-only permissions should not have option to create new document from diagram, document, presentation or spreadsheet template, see attached screenshot.

This is a fix for nextcloud v30.

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Documentation (manuals or wiki) has been updated or is not required
    shares-add-new-files-with-doc-creation

@tanyaka tanyaka marked this pull request as ready for review May 26, 2025 10:28
@tanyaka tanyaka changed the title IONOS: fix: for shares allow document creation only for users that ca… fix: for shares allow document creation only for users that ca… May 26, 2025
@juliusknorr juliusknorr added bug Something isn't working 3. to review Ready to be reviewed labels May 27, 2025
@tanyaka tanyaka force-pushed the fix/shares-disable-doc-creation-v30-shares branch from 579b30e to 14f78c1 Compare May 27, 2025 08:07
@elzody elzody changed the base branch from main to stable30 May 28, 2025 22:17
@elzody elzody changed the base branch from stable30 to main May 28, 2025 22:18
@elzody
Copy link
Copy Markdown
Collaborator

elzody commented May 28, 2025

If this is a fix for Nextcloud v30 only, we would need to change the base branch from main to stable30 and then you would need to rebase onto stable30 and force push from your local branch. Or do I misunderstand?

@tanyaka
Copy link
Copy Markdown
Author

tanyaka commented Jun 2, 2025

If this is a fix for Nextcloud v30 only, we would need to change the base branch from main to stable30 and then you would need to rebase onto stable30 and force push from your local branch. Or do I misunderstand?

My bad, this is probably also a fix for v29, it's just not for v31, this file is not a part of v31 any more.

Edited: discussed with artonge, clarified.

@artonge artonge changed the base branch from main to stable30 June 3, 2025 08:50
@artonge artonge changed the base branch from stable30 to main June 3, 2025 08:51
@artonge
Copy link
Copy Markdown
Contributor

artonge commented Jun 3, 2025

git reset --hard HEAD~600
git rebase origin/stable30
git cherry-pick 14f78c1fde1d7879250092f93b5bd9306c8b6465

@artonge artonge changed the title fix: for shares allow document creation only for users that ca… [stable30] fix: for shares allow document creation only for users that ca… Jun 3, 2025
@artonge
Copy link
Copy Markdown
Contributor

artonge commented Jun 3, 2025

/backport to stable29

@tanyaka tanyaka force-pushed the fix/shares-disable-doc-creation-v30-shares branch from 14f78c1 to 090e41e Compare June 3, 2025 10:35
@tanyaka tanyaka changed the base branch from main to stable30 June 3, 2025 10:38
…n edit

Signed-off-by: Franziska Bath <franziska.bath@strato.de>
@tanyaka tanyaka force-pushed the fix/shares-disable-doc-creation-v30-shares branch from 090e41e to b5b88f3 Compare June 3, 2025 10:43
@elzody
Copy link
Copy Markdown
Collaborator

elzody commented Jun 4, 2025

Seems fine to me now, thanks for the rebase. The failing Cypress tests are unrelated to this pull request, and have already been fixed, so this can safely be merged.

@elzody elzody merged commit 742a75a into nextcloud:stable30 Jun 4, 2025
51 of 58 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Ready to be reviewed bug Something isn't working feedback-requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants