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

api: rethink user identity #105

Open
alejandromumo opened this issue Jul 3, 2023 · 0 comments
Open

api: rethink user identity #105

alejandromumo opened this issue Jul 3, 2023 · 0 comments

Comments

@alejandromumo
Copy link
Member

          This feels a bit out of place here. 

The problem was that to publish a record, we needed the user's identity. Therefore, we used a utility function from invenio-access to generate this identity given the user id.

But, there is permission needed to publish a draft (authenticated_user) and it was added here.

Possible alternatives:

1 - move this to invenio-rdm-records as this is a specific RDM action (publish)
2 - move this to invenio-access as a new utility function (e.g. get_authenticated_identity)

Originally posted by @alejandromumo in #99 (comment)

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

No branches or pull requests

1 participant