Skip to content

wip #184

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

Closed
Closed

Conversation

utnapischtim
Copy link
Contributor

@utnapischtim utnapischtim commented Feb 24, 2025

that pr should enable the possibility to run something like
invenio apply "roles create administration-moderation --description oo && roles create administration" with one app context

i gave up to implement following:

invenio apply "access allow administration-moderation role administration-moderation && access allow administration-access role administration"

(at least for the moment):
the problem with the access allow is that allow is a subgroup of access which makes it harder to find the ctx to invoke from. and i think the resultcallback here thing in invenio-access makes it also not easier.

ps.: it would be easier to refactor the invenio-access.cli code for me it is difficult to understand what invenio access allow administration-moderation role administration-moderation really does

@utnapischtim utnapischtim marked this pull request as draft February 24, 2025 22:45
@utnapischtim
Copy link
Contributor Author

close in favor of inveniosoftware/invenio-app#99

@utnapischtim utnapischtim deleted the add-apply-command branch February 27, 2025 10:25
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.

1 participant