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

Add Folder Sync Connection sometimes does not warn #7741

Closed
jnweiger opened this issue Feb 12, 2020 · 2 comments
Closed

Add Folder Sync Connection sometimes does not warn #7741

jnweiger opened this issue Feb 12, 2020 · 2 comments
Assignees
Milestone

Comments

@jnweiger
Copy link
Contributor

The GUI has a prominent button Add Folder Sync Connection (that should never be used IMHO).

If the user adds a second Folder Sync Connection, he normally gets a warning,
image
This warning is good, as there is no option to replace the '/' connection with another one. So every second connection is an overlap.
After accepting the warning, the connection is created, and the selected subfolder is synced outside of the main tree again.

The user can now delete the original Folder Sync Connection
image

... and recreate a sync connection for the parent folder:

image

This time, it is again an overlap, but no narning is shown.
The warning should be shown in both cases.

@ogoffart ogoffart self-assigned this Feb 19, 2020
@ogoffart ogoffart added the ReadyToTest QA, please validate the fix/enhancement label Mar 5, 2020
@ogoffart ogoffart added this to the 2.6.2 milestone Mar 5, 2020
@jnweiger jnweiger mentioned this issue Sep 10, 2020
63 tasks
@TheOneRing TheOneRing removed the ReadyToTest QA, please validate the fix/enhancement label Sep 10, 2020
@TheOneRing TheOneRing modified the milestones: 2.6.2, 2.7.0 Sep 10, 2020
@TheOneRing
Copy link
Member

The issue is part of the changelog and will be tested in the qa process, reopen if the issue was not fixed.

@jnweiger
Copy link
Contributor Author

jnweiger commented Nov 9, 2020

confirmed fixed in 2.7.0 rc3

The expected Warning: You are already syncing /Photos/, which is a subfolder of /. is shown.

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

No branches or pull requests

3 participants