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

In test can't update resources on packages created before scheming migration #814

Open
jachurchill opened this issue Nov 8, 2019 · 0 comments
Assignees
Labels

Comments

@jachurchill
Copy link
Contributor

jachurchill commented Nov 8, 2019

Currently, when a user updates a single resource, all existing resources are validated (this is a core CKAN functionality). If any resource is invalid, then it's impossible to update another.

Many resources (especially geographic datasets) are not validating because mandatory fields are not set to any value. We need to set these mandatory fields properly to allow for proper validation.

To fix this we will need to:

  • set reasonable default values; and
  • update the migration script to support this

NB - If we extend the new schema to add additional mandatory fields, we will have to use this workflow again to ensure proper validation (and thus proper updating).

@jachurchill to also create a script that will allow us to extend the new schema with additional mandatory fields.

@jachurchill jachurchill added the bug label Nov 8, 2019
@jachurchill jachurchill self-assigned this Nov 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant