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

Windows: Display name for folder sync pairs does not include remote path #8104

Closed
TheOneRing opened this issue Sep 21, 2020 · 6 comments
Closed

Comments

@TheOneRing
Copy link
Member

TheOneRing commented Sep 21, 2020

Current:

image

New:

image
image
image

@michaelstingl
Copy link
Contributor

@TheOneRing oCIS? Should be fixed in latest version. (display-name vs displayname)

@TheOneRing
Copy link
Member Author

no thats unrelated to the wrong user name, I'm talking about multiple sync folder pairs.
here / and /sigh/

@michaelstingl
Copy link
Contributor

I'd follow more the Federated Cloud ID:
test@demo.owncloud.com
(https://demo.owncloud.com/settings/personal)

Then add the full path or only the last path segment maybe…

CleanShot 2020-09-21 at 13 44 09

@pmaier1 what do you think how to display the left sidebar in Win Explorer?

@michaelstingl
Copy link
Contributor

Alternative would be, to use the name of the local folder the user choose:

CleanShot 2020-09-21 at 14 37 18

@pmaier1
Copy link

pmaier1 commented Sep 21, 2020

I basically like the idea in the example and I also like the idea of using the Fed ID here. I'd say we do one of the following:

a) Like today but with sync connection location

ownCloud - <domain> (<user>) - <sync connection location>

b) With Fed ID and sync connection location

ownCloud - <user@domain> - <sync connection location>

=> ownCloud - demo@cloud.owncloud.com - /Documents

I'd like to see b) in action. Actually we should embrace federation but not sure if it maybe looks too technical here. We might rather want to use the display name.
Maybe we can drop the last part (- <sync connection location>) if it's only one sync connection and that's /. I'd expect that mostly power users will have more than one sync connection and those will understand.

For the sync folder naming default, we could go for

<user@domain> - <sync connection location>

As slashes are probably not allowed (is @ allowed? 🤔 ), we could just mention the remote folder that equals the sync connection (without path). I don't think we necessarily have to mention "ownCloud" here again.

What do you think?

@TheOneRing
Copy link
Member Author

TheOneRing commented Oct 1, 2020

How about:
image

The test pilot one is old scool

TheOneRing added a commit to TheOneRing/client that referenced this issue Oct 2, 2020
@TheOneRing TheOneRing changed the title Windows: Display name for folder sync pairs does not include remote path Adjust display name to improve its syntactical value Oct 2, 2020
@TheOneRing TheOneRing changed the title Adjust display name to improve its syntactical value Windows: Display name for folder sync pairs does not include remote path Oct 2, 2020
TheOneRing added a commit to TheOneRing/client that referenced this issue Oct 2, 2020
This was referenced Oct 13, 2020
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