inveniosoftware / invenio-records-permissions

Permissions for Invenio's records REST API.
https://invenio-records-permissions.readthedocs.io
MIT License
0 stars 25 forks source link

Introduce conditional `DisableIf` generator & read-only mode #86

Open max-moser opened 1 year ago

max-moser commented 1 year ago

This can be used to implement a conditional read-only mode, e.g. enabled temporarily to make system migrations smoother.

To do:

fenekku commented 1 year ago

Coordination:

This one has been sitting for 3weeks+

Surfacing this back to attention for @utnapischtim , @ppanero and @ntarocco . If someone else would be better suited to look at it, let us know!

chokribr commented 1 year ago

@utnapischtim @ppanero please take a look!

max-moser commented 6 months ago

Given that Mr. E already provided the ConditionalGenerator, I reworked this PR to be based off that instead. Over the course of that, the functionality of having the config option a callable got thrown out the window.

Given that we override most of the permission policies in our own modules anyway, we can add that feature in locally for us.