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

Desktop Sharing / Desktop Overlay Icons - show shared by me files #3043

Closed
stonerl opened this issue Mar 30, 2015 · 14 comments
Closed

Desktop Sharing / Desktop Overlay Icons - show shared by me files #3043

stonerl opened this issue Mar 30, 2015 · 14 comments
Milestone

Comments

@stonerl
Copy link

stonerl commented Mar 30, 2015

When a file is shared to me by another user, the file-manager (e.g. Finder on OS X) indicates this by "extending" the overlay with the sharing icon.
screen shot 2015-03-30 at 11 03 48
But files I shared with other users or by link are not indicated at all (the upper file in the picture). Would it make sense to add such overlay icons?

Furthermore since desktop sharing has been introduced, imho it would be quite handy to have the possibility to see the shares in the desktop client as it can be on the server (Shared with you / Shared with others / Shared by link).

@guruz
Copy link
Contributor

guruz commented Mar 31, 2015

This might actually a server bug..
Which server version do you use? I remember there was something related to "Shared" flag being sent down to client as file permission.

CC @icewind1991 @PVince81

@guruz guruz added this to the 1.8.1 - Bugfix milestone Mar 31, 2015
@stonerl
Copy link
Author

stonerl commented Mar 31, 2015

1.8.0 stable

@guruz
Copy link
Contributor

guruz commented Mar 31, 2015

@stonerl Server, not client :)

@stonerl
Copy link
Author

stonerl commented Mar 31, 2015

@guruz Sorry :-). 8.0.2

@mihaig11
Copy link

mihaig11 commented Apr 7, 2015

I have exactly the same problem with the linux 1.8.0 client and 7.0.5 server.

@danimo
Copy link
Contributor

danimo commented Apr 7, 2015

@guruz This is not a server bug. It's a true limitation. Problem is again how to reflect it on Windows. All other OSes can have those icons easily I think.

@guruz
Copy link
Contributor

guruz commented Apr 8, 2015

@danimo This is about

But files I shared with other users or by link are not indicated at all (the upper file in the picture).

with other vs from other

@icewind1991 @PVince81 Any comment? :)

@ogoffart ogoffart removed this from the 1.8.1 - Bugfix milestone Apr 8, 2015
@ogoffart
Copy link
Contributor

ogoffart commented Apr 8, 2015

We are showing the icon if the server report a "S" bit in the permission field.
Most likely the server don't report the right sharing state.

@Dianafg76
Copy link

I tried the problem in Windows, desktop v 1.8.2 (build 5133 ), Server v{"installed":true,"maintenance":false,"version":"8.1.0.6","versionstring":"8.1 beta 2","edition":"Enterprise"} and passes the same

@dragotin
Copy link
Contributor

dragotin commented Jun 2, 2015

Yes, that works as designed: Files that are shared by me are not marked as shares. There have been discussions earlier, but that is the status quo IIRC.

@rullzer
Copy link
Contributor

rullzer commented Mar 24, 2016

Now that owncloud/core#22789 is in (and backported to 9.0.1). We could implement this on the client.

Basically ask for the webdav property share-types and you get the info you want.

@guruz
Copy link
Contributor

guruz commented Apr 26, 2016

@rullzer @icewind1991 This means during discovery we always need to ask 1 more property. Which is OK but not nice.
I'm curious, what was the reason for not supplying this info also via S permission?

@guruz guruz removed this from the 2.3.0-next milestone Apr 26, 2016
@guruz guruz changed the title Desktop Sharing / Desktop Overlay Icons - show shared files Desktop Sharing / Desktop Overlay Icons - show shared by me files Apr 26, 2016
@guruz guruz added this to the 2.3.0-next milestone Apr 26, 2016
@rullzer
Copy link
Contributor

rullzer commented Apr 26, 2016

Well S was defined to be shared with you as far as I know.
We could extend this to also include yet another letter to indicated shared by the current user. The new property was chosen because it is cleaner. And we can directly indicate how a file was shared. Which is something clients can chose to do.

@dragotin
Copy link
Contributor

duplicate of #4788

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

9 participants