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

services: fix permission for file edit. #1746

Merged

Conversation

alejandromumo
Copy link
Member

@slint slint merged commit 6d3942e into inveniosoftware:master May 17, 2024
4 checks passed
@slint
Copy link
Member

slint commented May 21, 2024

@tmorrell I remember you mentioned using this function/config. The issue was that the Draft was being passed as the record parameter, and the Record wasn't passed at all, so we've added more explicit params and switched the values passed to be correct. Depending on what kind of attributes you were checking this might not affect you at all, but just a heads up.

@tmorrell
Copy link
Contributor

Thanks @slint, I've updated our test instance. You might want to update https://github.com/inveniosoftware/docs-invenio-rdm/blob/dev/docs/customize/files-and-versioning.md as well.

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.

edit files: allow users to modify files for non-zenodo DOIs
3 participants