Skip to content

Conversation

jessegeens
Copy link
Contributor

@jessegeens jessegeens commented Sep 16, 2025

To be able to show all shares to project admins, we stop injecting a filter for the current user, and rely on calling functions to put a user filter in the request filters

@jessegeens jessegeens changed the title WIP WIP: show shares to project admins Sep 16, 2025
@jessegeens
Copy link
Contributor Author

To be merged together with cs3org/reva#5301

@jessegeens jessegeens force-pushed the feat/proj-admin-shares branch 2 times, most recently from 114a39a to 6a97f29 Compare September 17, 2025 14:30
@jessegeens jessegeens changed the title WIP: show shares to project admins Show shares to project admins Sep 17, 2025
@jessegeens jessegeens marked this pull request as ready for review September 17, 2025 14:31
jessegeens added a commit to cs3org/reva that referenced this pull request Sep 17, 2025
This auto-injected filter was removed in the plugins by cernbox/reva-plugins#51
@jessegeens jessegeens force-pushed the feat/proj-admin-shares branch 2 times, most recently from f5b5dbc to 51a0380 Compare September 18, 2025 13:05
This is to allow project admins to see all shares
on a resource
Copy link
Member

@glpatcern glpatcern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the "easy" part, still have to go through the Reva PR!

@glpatcern glpatcern merged commit f2c6459 into master Sep 23, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants