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

Bugfix/do not warn about invalid files in not synced folders #7977

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mgallien
Copy link
Collaborator

@mgallien mgallien commented Mar 6, 2025

No description provided.

@mgallien mgallien added this to the 3.16.1 milestone Mar 6, 2025
mgallien added 2 commits March 7, 2025 12:03
we currently notify invalid file names for files that the sync engine
will not upload because they are ignored for other reasons (selective
sync for example)

let's first check if the file is ignored and only if the file would be
uploaded, we can check the file name validity

Signed-off-by: Matthieu Gallien <[email protected]>
if a file name is invalid, the files will be ignored

if a file is excluded because of selective sync, the file will be
ignored

if a file is ignored for those both reasons, avoid showing a very
visible warning to the user

will limit too many warnings and notifications being shown

Signed-off-by: Matthieu Gallien <[email protected]>
@mgallien mgallien force-pushed the bugfix/doNotWarnAboutInvalidFilesInNotSyncedFolders branch from b37baf4 to 6234375 Compare March 7, 2025 12:00
Copy link

github-actions bot commented Mar 7, 2025

Artifact containing the AppImage: nextcloud-appimage-pr-7977.zip

SHA256 checksum: 848252cfc239089867613df3d2625a97ff0b6489ac361bd23fa678307cdf7ee9

To test this change/fix you can download the above artifact file, unzip it, and run it.

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

Copy link

sonarqubecloud bot commented Mar 7, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
E Maintainability Rating on New Code (required ≥ A)
43 New Code Smells (required ≤ 0)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@mgallien mgallien requested a review from claucambra March 7, 2025 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants