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

Support Groupfolders #129

Closed
xf- opened this issue Jan 18, 2020 · 9 comments · Fixed by #398
Closed

Support Groupfolders #129

xf- opened this issue Jan 18, 2020 · 9 comments · Fixed by #398
Labels
3. to review Waiting for reviews discussion Being discussed enhancement New feature or request

Comments

@xf-
Copy link

xf- commented Jan 18, 2020

We use group folders app for shared space and most apps support them without any issue, but new photo app won't show any folders/images.

@Mikescops Mikescops added the 0. Needs triage Pending approval or rejection. This issue is pending approval. label Jan 19, 2020
@skjnldsv
Copy link
Member

cc @rullzer @icewind1991

@gabrielklein
Copy link

gabrielklein commented Jan 29, 2020

Same problem. I have a "group folder" with all my pictures. I share it with my wife.

When I do a
php -f occ maps:scan-photos

Only my "personal folder" is scanned.
/data/gabriel/files

The "group folder" is in my case in this location.
/data/__groupfolders/1

It is not scanned and not displayed (on the map).

It's difficult to know if this issue should be reported here or on "maps" - because I don't see the picture of my shared folder too. :)

@gabrielklein
Copy link

Probably related to nextcloud/maps#123

@jakobroehrl
Copy link
Contributor

Unfortunately not possible to implement due to performance issues :(

@svenb1234
Copy link

What difference does it make whether a user's photo folder has 30.000 photos or a shared folder has 30.000 photos? Either the performance is up to job of handling large collections or not. Due to Smartphones and the ease of taking pictures these numbers are normal and Synology as well as Qnap have no issue in handling them. Neither in the web client nor the Android app. Yet both do not work well for Nextcloud.

@skjnldsv
Copy link
Member

@svenb1234 depends on the technical implementation of the backend.

@rullzer
Copy link
Member

rullzer commented Jul 21, 2020

Let me take a step back. What part are you looking to get groupfolders? Getting everything in your normal blogroll is hard. Having it in the shared part might be possible indeed.

@xf-
Copy link
Author

xf- commented Jul 21, 2020

@rullzer instead of sharing every photofolder form an event (and multiple people took photos), we use a shared folder to share them. Advantage to decrease the personal storage and increase the public/internal storage for shared stuff. Even a extra section with group folders would be fine, but since replacing the gallery, you need to go trough all the folders instead of using the image browser (now photos).

@darmbrust
Copy link

darmbrust commented Jul 23, 2020

I'm not sure if they way I use nextcloud is the same as "group folders" - but without the ability for photos to show photos from shared folders, its useless to me.

I have an account that is only used by my phone to upload photos.

The folders that the phone uploads to are shared with my main account.

Photos ignores that those folders even exist, making it useless.

I also can't understand the logic that this can't be done due to "performance" issues. Its another folder on the local server I have permissions to. Sounds more like a design problem....

It doesn't even show me photos I mark as "favorite" if they came from a shared folder. Which is confusing...

@skjnldsv skjnldsv added 3. to review Waiting for reviews and removed 0. Needs triage Pending approval or rejection. This issue is pending approval. labels Jul 24, 2020
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews discussion Being discussed enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants