-
Notifications
You must be signed in to change notification settings - Fork 685
[Share] As an OC user I want to be able to share data with ownCloud users #425
Comments
@rperezb @javiergonzper @ggdiez @nasli @purigarcia looks good in general, but please use iOS 8 as base for your mockups? :D |
@jancborchardt yes 😵 it was the tool, http://pencil.evolus.vn/ it is not updated... |
Hi @jancborchardt @rperezb @nasli @purigarcia @MTRichards @cmonteroluque Using the Raquel idea these are my mockups. The structure of the view is the same but I have done some changes.
My mockup: Shared 0 view, it's the view when the user tap in share option on a not shared file. Shared 1 view, it's when the user have share the file with an user and shared by link with password and password protect options. Find user view, it's when the user tap in "Add User or Group" button. In this view the user can search for the user/users or group/groups to share the file and the permissions. |
I like the latest three mockups. |
@cmonteroluque @MTRichards @PVince81 @DeepDiver1975 |
@tobiasKaminsky for the first iteration the part of the permissions is not included. So the mockup of the third image will be the same without the "options" section. But I'm thinking in a solution for the next iteration. ;) Thanks |
@jancborchardt pointed me to this issue as I am developing for android and he said it would be good that both apps have a similar behaviour. So, do you think it would work with a popup? With this the list of the user would not be truncated by the options menu. |
We can simplify this. The »Share with user or group …« section should not be a popup. It should be much like the web interface dropdown, with the »can edit« and »can share« sections in the same line of the username, to the right. I’ll work on a mockup. |
I have checked the Share API documentation and there are no way to know if a file shared by link have password protected enabled. As @schiesbn said me the only way for the moment is use the "share_with" column in the response. It's ugly but there are not other way. |
@ggdiez WIP the server API owncloud/core#16646 |
related to the desktop client: owncloud/client#3537 and android: owncloud/android#880 |
A mockup would be great. |
consolidated server api spec is here - owncloud/core#18117 |
Any News about this work ? :D |
Work in progress, first part is the OCS- Api, then we will update the iOS client. |
^^ FYI @dragotin @ogoffart @rullzer @jturcotte etc if we do this in the client, it could look similar |
Yes, it would be great! Indeed, you are welcome to the mobile and pals room @ownclouders conference, we may talk about it and the flow 😄 |
BTW: As live mockup essentially, please look at how it is done on the server – this is always the master regarding design. For specific optimizations it will be mostly showing stuff directly (to not need to type on mobile), but we already talked about that @rperezb :) |
Yes, we talked about it, mostly we will follow #425 (comment) |
For the next version, 3.4.5, the first step is done, add a new share view and add additional options for the sharelink (expiration date and password) |
Share with users in progress on: https://github.com/owncloud/ios/tree/sharee_with_user |
@rperezb @ggdiez @javiergonzper @nasli show screenshots as soon as you can please. :) |
This is done and it will be available within the next ownCloud iOS versions |
AC:
NOTE:
@MTRichards @jancborchardt @cmonteroluque @javiergonzper @ggdiez @nasli @purigarcia
The text was updated successfully, but these errors were encountered: