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

renamed use restriction enums. #245

Merged
merged 3 commits into from
Jun 29, 2021
Merged

Conversation

pinarpink
Copy link
Member

Fixes #230

@pinarpink pinarpink requested a review from vildead June 22, 2021 18:27
Copy link
Member

@vildead vildead left a comment

Choose a reason for hiding this comment

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

Nice! The I have noticed the MR in JSON schemas is also ready ;)
Just minor: There are two migration files. Could we squish them into one?

@@ -15,7 +15,7 @@ class DatasetsImporter(BaseImporter):
"""

json_schema_validator = DatasetJSONSchemaValidator()
json_schema_uri = 'https://raw.githubusercontent.com/elixir-luxembourg/json-schemas/master/schemas/elu-dataset.json'
#json_schema_uri = 'https://raw.githubusercontent.com/elixir-luxembourg/json-schemas/master/schemas/elu-dataset.json'
Copy link
Member

Choose a reason for hiding this comment

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

@jLebioda what do you think? Can I include the repository with JSON schemas as a submodule? It makes more sense than having one url which might not work and another fallback copy of schemas in fixtures.

@pinarpink
Copy link
Member Author

pinarpink commented Jun 29, 2021

Nice! The I have noticed the MR in JSON schemas is also ready ;)
Just minor: There are two migration files. Could we squish them into one?

@vildead I merged migrations added migration of existing data also added the exposure of use category labels to JSON (fix for #250)

@pinarpink pinarpink merged commit 114ba11 into develop Jun 29, 2021
@pinarpink pinarpink deleted the 230-use-restriction-enum-rename branch July 20, 2021 10:31
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.

Use meaningful enumeration labels for "Use Restriction Rule"
2 participants