-
Notifications
You must be signed in to change notification settings - Fork 28
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
Draft edit action #452
Draft edit action #452
Conversation
ppanero
commented
Aug 11, 2023
•
edited
Loading
edited
- requires actions: add edit draft inveniosoftware/invenio-rdm-migrator#136
- closes migration-actions: Record/draft #428
blocked until we have a db fixture able to rollback changes |
d7867db
to
946b470
Compare
unblocked, code fixed and tests passing in local |
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.
LGTM, and my question from the invenio-rdm-migrator
PR got answered :)
# no need to differentiate from record update since we wont do that in support | ||
# and user edits (e.g. web ui) pass by a draft (create + [edit] + publish actions) |
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.
comment: indeed that is the case! scrape my comment regarding what I said in the invenio-rdm-migrator
PR, edits and updates can be handled in the same action.
a02303d
to
0c7b89c
Compare