-
Notifications
You must be signed in to change notification settings - Fork 84
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
mappings: migrate resource_type to vocbulary style #623
Conversation
261ba86
to
da5819d
Compare
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, I knew there were going to be resource_types remnants.
Leaving previous versions alone is important, approving though so you are unblocked when the tests are fixed.
invenio_rdm_records/records/mappings/v6/rdmrecords/drafts/draft-v2.0.0.json
Outdated
Show resolved
Hide resolved
invenio_rdm_records/records/mappings/v6/rdmrecords/records/record-v2.0.0.json
Outdated
Show resolved
Hide resolved
invenio_rdm_records/records/mappings/v7/rdmrecords/drafts/draft-v2.0.0.json
Outdated
Show resolved
Hide resolved
invenio_rdm_records/records/mappings/v7/rdmrecords/records/record-v2.0.0.json
Outdated
Show resolved
Hide resolved
system_identity, "resource_types", "rsrct") | ||
|
||
|
||
@pytest.fixture(scope="function") |
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.
Had to make it function scoped and therefore remove it from runnint_app
. The issue comes when used along es_clear
. Since the vocabulary will be indexed and the labels come from ES, any previous test with es_clear
will remove it and it will not be present when needed (e.g. labels). Note: this only happens at tests/resources/
level since es_clear is not used otherwise.
resource_type_facet = NestedTermsFacet( | ||
field='metadata.resource_type.type', | ||
subfield='metadata.resource_type.subtype', | ||
resource_type_facet = TermsFacet( |
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.
triggers #628
closes #622
Facets are back