nextcloud / files-clients

1 stars 0 forks source link

Make groupfolders more visible #38

Closed tobiasKaminsky closed 1 year ago

tobiasKaminsky commented 1 year ago

Goal is to give our groupfolder system more visibility.

Android @tobiasKaminsky:

Desktop @mgallien:

iOS @marinofaggiana:

mgallien commented 1 year ago

@tobiasKaminsky do you have a mock up of what it would look like ? we currently handle them like external storage so if they are selected the activity from them would already appear in the activity list maybe I misunderstand something here

tobiasKaminsky commented 1 year ago

image

It is about this view and about "folder" icon (the one with green sync icon). On Desktop it is basically to have a shortcut to those groupfolders.

marinofaggiana commented 1 year ago

Is in WIP the capabilities: groupfolder for user: yes/no ?? because in NC26 is not present (yet)

tobiasKaminsky commented 1 year ago

Will be at some point in groupfolders app, and then we can use it. Might be backported to NC26, but clients shall only rely on capability.

marinofaggiana commented 1 year ago

Simulator Screenshot - iPhone 14 - 2023-04-13 at 15 26 42 Simulator Screenshot - iPhone 14 - 2023-04-13 at 15 26 48

I would propose the addition in share (like server)

Screenshot 2023-04-13 alle 15 27 47
tobiasKaminsky commented 1 year ago

Let us first do the minimum, and if time allows, we can add this to share handling.

tobiasKaminsky commented 1 year ago
},
                "groupfolders": {
                    "hasGroupFolders": true
                },

We have on e2e.kaminsky.me, user "tag" this now active. So we can do:

tobiasKaminsky commented 1 year ago

@marinofaggiana @mgallien ^

marinofaggiana commented 1 year ago

@tobiasKaminsky: in my NC26 for NOT admin:

GET https://nextcloud.homelinux.org/index.php/apps/groupfolders/folders error 500 < {"message":"Logged in user must be an admin, a sub admin or gotten special right to access this setting"} >

as admin group, no problem.

tobiasKaminsky commented 1 year ago

This is not yet in NC26, thus please use e2e.kaminsky.me server, where all needed PRs are applied.

tobiasKaminsky commented 1 year ago

@marinofaggiana @mgallien in request to list groupfolders please add applicable=1, as this then shows also for admin user correct info: /index.php/apps/groupfolders/folders?applicable=1

tobiasKaminsky commented 1 year ago

@jancborchardt when we first spoke about this feature, you mentioned that icon might change. So, will we have a new icon? :)

tobiasKaminsky commented 1 year ago

Capability:

    "groupfolders": {
                    "hasGroupFolders": true
                },

List of groupfolders: curl --request GET \ --url 'https://e2e.kaminsky.me/index.php/apps/groupfolders/folders?format=json&applicable=1' \ --header 'OCS-APIRequest: true'

output:

{
    "ocs": {
        "meta": {
            "status": "ok",
            "statuscode": 100,
            "message": "OK",
            "totalitems": "",
            "itemsperpage": ""
        },
        "data": {
            "1": {
                "id": 1,
                "mount_point": "\/groupfolder",
                "groups": {
                    "users": 31
                },
                "quota": -3,
                "size": 1896240,
                "acl": false,
                "manage": []
            },
            "2": {
                "id": 2,
                "mount_point": "\/subfolder\/group",
                "groups": {
                    "users": 31
                },
                "quota": -3,
                "size": 2604768,
                "acl": false,
                "manage": []
            }
        }
    }
}
tobiasKaminsky commented 1 year ago

All done.