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

Disable download from albums #1641

Merged
merged 2 commits into from
Feb 21, 2023
Merged

Conversation

artonge
Copy link
Collaborator

@artonge artonge commented Feb 21, 2023

Temporary fix for #1394

@artonge artonge added bug Something isn't working 3. to review Waiting for reviews javascript Javascript related ticket feature: albums Related to the albums section labels Feb 21, 2023
@artonge artonge added this to the Nextcloud 26 milestone Feb 21, 2023
@artonge artonge requested a review from a team February 21, 2023 11:54
@artonge artonge self-assigned this Feb 21, 2023
@artonge artonge requested review from skjnldsv, Pytal and szaimen and removed request for a team February 21, 2023 11:54
@szaimen
Copy link
Contributor

szaimen commented Feb 21, 2023

@artonge I suppose the correct fix would be much work?

@artonge
Copy link
Collaborator Author

artonge commented Feb 21, 2023

@artonge I suppose the correct fix would be much work?

Yes, sadly, it would involve a plugin system to download files, as the current download API is not able to check whether a file is accessible from an album.

Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

All right! LGTM for now then but didnt test

@artonge artonge force-pushed the artonge/fix/disable_download_from_album branch from 3306e01 to 06a342d Compare February 21, 2023 12:03
Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge artonge force-pushed the artonge/fix/disable_download_from_album branch from 06a342d to d33bf56 Compare February 21, 2023 15:18
Signed-off-by: Louis Chemineau <louis@chmn.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug Something isn't working feature: albums Related to the albums section javascript Javascript related ticket
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants