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

Don't treat new folders in unpinned folders as files to dehydrate #2868

Conversation

er-vin
Copy link
Member

@er-vin er-vin commented Jan 26, 2021

This would only happen if the parent of the newly created folder would
be explicitly set to online only, hence why it went under the radar
previously.

Signed-off-by: Kevin Ottens kevin.ottens@nextcloud.com

@er-vin
Copy link
Member Author

er-vin commented Jan 26, 2021

/rebase

@github-actions github-actions bot force-pushed the vfs_cfapi_dont_mistake_unpinned_directory_for_files_to_dehydrate branch from 89485d7 to 62a926f Compare January 26, 2021 17:07
@er-vin
Copy link
Member Author

er-vin commented Jan 26, 2021

/rebase

This would only happen if the parent of the newly created folder would
be explicitly set to online only, hence why it went under the radar
previously.

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>
@github-actions github-actions bot force-pushed the vfs_cfapi_dont_mistake_unpinned_directory_for_files_to_dehydrate branch from 62a926f to ebfe51b Compare January 26, 2021 17:09
@nextcloud-desktop-bot
Copy link

AppImage file: Nextcloud-PR-2868-ebfe51b7e7b943cb220217222a4d1df7097fb314-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.

@er-vin er-vin merged commit 90a7f58 into master Jan 26, 2021
@er-vin er-vin deleted the vfs_cfapi_dont_mistake_unpinned_directory_for_files_to_dehydrate branch January 26, 2021 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants