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(preview): don't create folder structure when previews are disabled #45866

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

kesselb
Copy link
Contributor

@kesselb kesselb commented Jun 13, 2024

Summary

Don't create the folder structure for previews when previews are disabled.

TODO

  • CI
  • Review

Checklist

@kesselb kesselb added bug 3. to review Waiting for reviews labels Jun 13, 2024
@kesselb kesselb added this to the Nextcloud 30 milestone Jun 13, 2024
@kesselb kesselb self-assigned this Jun 13, 2024
Copy link
Member

@st3iny st3iny left a comment

Choose a reason for hiding this comment

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

Tested and works.

@kesselb kesselb requested review from susnux and come-nc June 13, 2024 14:14
@susnux
Copy link
Contributor

susnux commented Jun 13, 2024

works :)

@SystemKeeper
Copy link
Contributor

Failing tests should be fixed with #45883

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@susnux susnux force-pushed the bug/45697/disable-previews-two branch from 6905f95 to d270561 Compare June 18, 2024 08:39
@susnux susnux added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jun 18, 2024
@st3iny st3iny merged commit 250bb12 into master Jun 18, 2024
164 checks passed
@st3iny st3iny deleted the bug/45697/disable-previews-two branch June 18, 2024 11:57
@st3iny
Copy link
Member

st3iny commented Jun 19, 2024

Do we want to backport this?

@kesselb
Copy link
Contributor Author

kesselb commented Jun 19, 2024

/backport to stable29

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Disabling the preview does not stop the creation of the folder tree
4 participants