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

Show reason why group restriction not possible for that app #4864

Open
MorrisJobke opened this issue May 14, 2017 · 4 comments
Open

Show reason why group restriction not possible for that app #4864

MorrisJobke opened this issue May 14, 2017 · 4 comments
Labels
1. to develop Accepted and waiting to be taken care of enhancement feature: apps management good first issue Small tasks with clear documentation about how and in which place you need to fix things in. papercut Annoying recurring issue with possibly simple fix.

Comments

@MorrisJobke
Copy link
Member

Papercut from #2672

cc @eppfel @MariusBluem because you always asked for this

@MorrisJobke MorrisJobke added enhancement feature: apps management papercut Annoying recurring issue with possibly simple fix. labels May 14, 2017
@nextcloud-bot nextcloud-bot added the stale Ticket or PR with no recent activity label Jun 20, 2018
@wiswedel
Copy link
Contributor

@MorrisJobke Is this still of interest or can it be closed? Can't there be several reasons - none of them giving the reader any practical benefit?

@MorrisJobke
Copy link
Member Author

Can't there be several reasons - none of them giving the reader any practical benefit?

Yes there can be different and it would be helpful to communicate why given features are not there IMO.

@ghost ghost closed this as completed Jun 12, 2019
@skjnldsv skjnldsv added the 1. to develop Accepted and waiting to be taken care of label Jun 12, 2019
@skjnldsv skjnldsv reopened this Jun 12, 2019
@ghost ghost removed the stale Ticket or PR with no recent activity label Jun 12, 2019
@szaimen

This comment has been minimized.

@szaimen szaimen closed this as completed May 21, 2021
@szaimen szaimen reopened this May 27, 2021
@joshtrichards
Copy link
Member

I suppose these are the reasons:

canLimitToGroups(app) {
if ((app.types && app.types.includes('filesystem'))
|| app.types.includes('prelogin')
|| app.types.includes('authentication')
|| app.types.includes('logging')
|| app.types.includes('prevent_group_restriction')) {
return false
}
return true

<div v-if="app.active && canLimitToGroups(app)" class="app-details__actions-groups">

* Apps with these types can not be enabled for certain groups only
* @var string[]
*/
protected $protectedAppTypes = [
'filesystem',
'prelogin',
'authentication',
'logging',
'prevent_group_restriction',
];

Apps this applies to:

https://github.com/search?q=org%3Anextcloud+path%3A%2F%28%5E%7C%5C%2F%29info%5C.xml%24%2F+filesystem+OR+prelogin+OR+authentication+OR+logging+OR+prevent_group_restrictions&type=code

For many (most?) of these it's somewhat obvious why they can't be limited. But I can see how a few might be surprising (e.g. Collectives, Photos, Maps, Community Document Server, Office Online, Activity).

@joshtrichards joshtrichards added the good first issue Small tasks with clear documentation about how and in which place you need to fix things in. label Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of enhancement feature: apps management good first issue Small tasks with clear documentation about how and in which place you need to fix things in. papercut Annoying recurring issue with possibly simple fix.
Projects
None yet
Development

No branches or pull requests

6 participants