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

Objects without sharing features showing sharing options #3160

Closed
emersonkfuri opened this issue Oct 17, 2018 · 8 comments
Closed

Objects without sharing features showing sharing options #3160

emersonkfuri opened this issue Oct 17, 2018 · 8 comments

Comments

@emersonkfuri
Copy link

Description: Even if the objects (external storage and group folders) on web files app are explicit marked as non shareable, mobile app still shows share options

Steps to reproduce: Go on web app files, create group folders or external storages, disable sharing features for it, and view it on mobile

Version: 3.3.1 RC1

@nextcloud-android-bot
Copy link
Collaborator

GitMate.io thinks possibly related issues are #1029 (Feature Request?? Where is the option to view images without downloading??), #1773 (show pictures without downloading), #2443 (Improve uploading/sharing options), #33 (Feature Request: Add Wifi Only Options), and #994 (Opening a folder on an external share shows "no data available").

@AndyScherzinger
Copy link
Member

@tobiasKaminsky can you reproduce this? I don't have an IDE at hand at the moment but afaik we do not have a check for this in place but I might be wrong.

@emersonkfuri
Copy link
Author

emersonkfuri commented Oct 18, 2018

I was checking #3162 and it references about reshare options only in files. So maybe it is a regression because folders are showing too.

@tobiasKaminsky
Copy link
Member

@emersonkfuri in #3162 in the first screenshot, you can see that the sharing button is invisible for folders.
Isn't this what you expect?

@emersonkfuri
Copy link
Author

emersonkfuri commented Oct 18, 2018

Yes, it is! Just saying on #3162 the button is only hide for folders which is not happening right now.

And it should be hide for any object... 😀

@tobiasKaminsky
Copy link
Member

And it should be hide for any object

No, if you have a e.g.text file, you might want to send it to your email app. Then you want to click on the "share" button as this is a combined send/share.
(in this case sharing is then hidden):

2018-10-18-100026

@emersonkfuri
Copy link
Author

emersonkfuri commented Oct 18, 2018

But if a parent object is not shareable, why its children should be?

At least in my case, when I mark an external storage and/or a group folder to not be shareable, I expect that any objects inside it cannot be shared too.

@tobiasKaminsky
Copy link
Member

The share option is only meant for NC -> sharing with other user
Files can still be sent to other apps locally, even without NC: download a file, browse with any file manager to it and do whatever you want.

--> "share" is disabled, while "send" is still allowed

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

4 participants