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

[FEATURE] Require 'system_administrator' permission to '/plugins/caas_next_refid' endpoint #19

Open
lorawoodford opened this issue Jan 17, 2025 · 1 comment · May be fixed by #22
Open
Assignees
Labels
enhancement New feature or request

Comments

@lorawoodford
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
Extend required 'system_administrator' permission to existing '/plugins/caas_next_refid' endpoint per the following request: #17 (comment)

Describe the solution you'd like
Add permission.

Describe alternatives you've considered
N/A

Additional context
Request stemming from permissions structure on new endpoints added via #17

@lorawoodford lorawoodford added the enhancement New feature or request label Jan 17, 2025
@lorawoodford lorawoodford self-assigned this Jan 17, 2025
lorawoodford added a commit to lorawoodford/caas_aspace_refid that referenced this issue Jan 17, 2025
lorawoodford added a commit to lorawoodford/caas_aspace_refid that referenced this issue Jan 21, 2025
lorawoodford added a commit to lorawoodford/caas_aspace_refid that referenced this issue Feb 4, 2025
lorawoodford added a commit to lorawoodford/caas_aspace_refid that referenced this issue Feb 6, 2025
@lorawoodford
Copy link
Collaborator Author

Dragging this issue to "On Hold" given the difficulty encountered while working on it. Essentially, while adding in permissions to the backend controller works fine for those accessing this endpoint via the API, it does not work when the API call is triggered from the frontend form (aka, when a new ao is created or when an existing ao has the "regenerate ref id" checkbox checked) because the autogenerate action in the archival object model calls the API endpoint without session information on the frontend user that is making the request. This is likely solvable, but would require more work than there is time for given our desire to get the plugin into production.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
1 participant