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

feat: force all the dois updated with the form to become schema 4 #452

Merged
merged 4 commits into from
Aug 24, 2020

Conversation

kjgarza
Copy link
Contributor

@kjgarza kjgarza commented Jul 29, 2020

Purpose

Forces all DOIs updated in the form to become schema 4.

closes: #447

Approach

All DOIs updated in the form will be transformed to schema 4. Independently of the state of the DOI (draft, registered, findable)

Learning

Looking back at the form support messages it can be seen that the form has not support schema transformation on the past. See https://app.frontapp.com/open/msg_29opdsb from 2019 ,which is related to datacite/datacite#654

This most likely due to the form modifying the metadata fields. The schema version is not a metadata field.

Status

  • Ready for Review

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@kjgarza kjgarza requested review from richardhallett and mfenner and removed request for richardhallett July 29, 2020 12:58
@kjgarza kjgarza self-assigned this Jul 29, 2020
@kjgarza kjgarza changed the title feat: force all the dois created with the form to be schema 4 feat: force all the dois updated with the form to become schema 4 Jul 29, 2020
@richardhallett
Copy link
Contributor

So what will happen in the scenario of older schema version with missing data, the form should fail to submit because it will be missing required fields right? and therefore once they fill it in, it will be valid schema 4?

@mfenner
Copy link
Contributor

mfenner commented Jul 29, 2020

Can't review as I am on vacation.

@kjgarza
Copy link
Contributor Author

kjgarza commented Jul 29, 2020

its mostly an FYI @mfenner .

@richardhallett yeah, in that case, it will fail until they fill all the fields correctly.

@mfenner mfenner removed their request for review July 29, 2020 14:39
@kjgarza kjgarza requested a review from mfenner August 3, 2020 13:22
Copy link
Contributor

@mfenner mfenner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. It would be good if the form would show a warning message if you open a DOI using a schema version < 4.

@kjgarza
Copy link
Contributor Author

kjgarza commented Aug 14, 2020

unrelated test are failing because DOIs already exist (422) which doesn't make sense. This might be related to datacite/lupo#606. and it happening even with the master branch and the last release. This needs be fixed to merge.

image

@kjgarza kjgarza merged commit 6b064ad into master Aug 24, 2020
@kjgarza kjgarza deleted the fix_force_version_4_for_form branch August 24, 2020 10:16
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.

The resourceTypeGeneral cannot be added to a DOI with schema 3 in the Form
3 participants