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

[Overlay] Once a file or folder is shared overlay icon is not updated #4785

Closed
mcastroSG opened this issue May 4, 2016 · 9 comments
Closed

Comments

@mcastroSG
Copy link

Expected behaviour

Once a file or folder is shared overlay icon should be updated to a "green shared" icon

Actual behaviour

Once a file or folder is shared the same "green check" icon remains

Steps to reproduce

  1. Right click over a file or folder
  2. Select "Share with ownCloud" option
  3. Pick an user or group and share with it the file

Server configuration

Server OS: Ubuntu
Server version: 9.1.0 Apha

Client configuration

OS X
Client OS: OS X El capitán 10.11.4
Client Version: Version 2.2.0-nightly20160504 (build 3331)

OS Linux
Client OS: Ubuntu 14.04 LTE
Client Version: Version 2.2.0nightly201654 (build 1587)

@mcastroSG mcastroSG added this to the 2.2.0-current milestone May 4, 2016
@dragotin
Copy link
Contributor

dragotin commented May 4, 2016

The share icon only indicates that a file was shared with me but currently not if it was shared by me.

@dragotin dragotin closed this as completed May 4, 2016
@rperezb
Copy link

rperezb commented May 4, 2016

@dragotin not sure, about it

If we have a look at the web:

  • when you share something with another user
    screen shot 2016-05-04 at 13 31 40
  • when someone shares something with you
    screen shot 2016-05-04 at 13 33 59

I mean, there are icons in both cases. Plus on the mobile side we follow this approach too

It'd be great if the desktop follows the same, as part of the parity goal 😃

Certainly, this is an enhancement for next versions, not the following one

@owncloud/designers @MTRichards @cmonteroluque what do you think?

@dragotin
Copy link
Contributor

dragotin commented May 4, 2016

I see. And I agree. But see explanation in owncloud/core#10247

How do the mobile clients query the information?

@jancborchardt
Copy link
Member

Also agree that the icon should definitely show in both cases.

cc @davivel @javiergonzper how do the mobipe clients do this, see @dragotin's question above.

@rperezb
Copy link

rperezb commented May 4, 2016

@dragotin we take advantage of the Share api to get those data shared BY YOU (https://doc.owncloud.org/server/8.2/developer_manual/core/ocs-share-api.html)

In the case of data shared with you it's taken from the propfind

@davivel
Copy link

davivel commented May 4, 2016

We make an extra call to the share API every time a folder is refreshed, calling GET [server]/ocs/v1.php/apps/files_sharing/api/v1/shares?path=[folder]&reshares=true&subfiles=true

This is done only when the user is browsing through the folders tree. In background refresh (full account sync in Android) updates of shares are ignored.

EDIT: fixed query values

@dragotin
Copy link
Contributor

dragotin commented May 4, 2016

ok, that wont work for us directly in the share api, but maybe we can store the share-by-me in the journal as a result of the discovery jobs.

Unfortunately that is nothing we can fix in the scope of 2.2.0.
@mcastroSG please create a feature request for improving this. Thanks!

@mcastroSG
Copy link
Author

mcastroSG commented May 4, 2016

@dragotin @rperezb #4788

@jancborchardt
Copy link
Member

Link somehow doesn't work, here's the correct one: #4788

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants