-
Notifications
You must be signed in to change notification settings - Fork 8
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
Implement updateFileMetadata use case #249
base: develop
Are you sure you want to change the base?
Implement updateFileMetadata use case #249
Conversation
02db5f7
to
2353a8f
Compare
…adata-on-file-page
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, looks good, just some minor comments.
@g-saracca Hi German, I finish my updates in this PR, please check it again. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ChengShi-1! Approving 👍🏼
What this PR does / why we need it:
Inspired from this TODO as well as the 2025 Q1 proposal. The File edit option for single file could be implemented since the api endpoints are ready here.
Which issue(s) this PR closes:
Related Dataverse PRs:
Special notes for your reviewer:
Suggestions on how to test this:
Visually inspect and make sure that tests pass
Is there a release notes update needed for this change?:
Additional documentation: