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

fix(ShareApiController): Set hideDownload to inverse of allow download attribute #48987

Closed
wants to merge 1 commit into from

Conversation

nfebe
Copy link
Contributor

@nfebe nfebe commented Oct 29, 2024

If the download attribute is set to false, then we should set hideDownload to true (essentially hiding the download button from the UI) and vice versa

Resolves: #46920

…d attribute

If the download attribute is set to false, then we should set hideDownload to
true (essentially hiding the download button from the UI) and vice versa

Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
@nfebe nfebe force-pushed the fix/46920/respect-no-download branch from ed1b327 to d9aa278 Compare October 29, 2024 18:04
@nfebe nfebe added 2. developing Work in progress 3. to review Waiting for reviews feature: sharing labels Oct 29, 2024
@nfebe
Copy link
Contributor Author

nfebe commented Oct 31, 2024

nextcloud/circles#1731 now comprehensively solves it.

@nfebe nfebe closed this Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing Work in progress 3. to review Waiting for reviews feature: sharing
Projects
Status: ☑️ Done
Development

Successfully merging this pull request may close these issues.

[Bug]: sharing with team does not respect "no download" setting
1 participant