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(keywords): Use category keywords, change category filter, change date filter #71

Merged
merged 5 commits into from
May 23, 2024

Conversation

Angi-Kinas
Copy link
Contributor

@Angi-Kinas Angi-Kinas commented May 22, 2024

This PR changes the keywords in the information block of the dataset page to use the keywords from the categories thesaurus (first level) and it changes the keywords on the search card to NOT use the categories thesaurus keywords, but all others (second level).
It also updates the "topic/thématique" filter to become a "category/categorie" filter and the date filter uses now the revisionYearForResource as the filter instead of the publication year.

image
image

to categories (first level thesaurus), change
keywords on search card to second level thesaurus
keywords
extend FieldsService to support categories
@Angi-Kinas Angi-Kinas force-pushed the change-keywords-to-categories-dataset-page branch from 85dfdc3 to 606bfdf Compare May 22, 2024 13:09
@Angi-Kinas Angi-Kinas changed the title feat(keywords): Change keywords on dataset page and search card feat(keywords): Use category keywords, change category filter, change date filter May 23, 2024
Copy link
Member

@tkohr tkohr left a comment

Choose a reason for hiding this comment

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

Thanks for this nice feature @Angi-Kinas and for making the date filtering consistent!

Everything works well on my side. I guess it's normal that we currently only have three values in the categories filter on integration, since not more terms of the thesaurus have been attached to records yet?

})
)
searchConfig = [
'categoryKeyword',
Copy link
Member

Choose a reason for hiding this comment

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

Just wondering if category would be sufficient here, as it is also part of the URL and would align with the filter label. Even though I agree that categoryKeyword is technically more explicit. I let you judge.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point 👍 , I think both works, but when I look at gn-ui where the inspireKeyword was extracted in a similar way, I will keep the Keyword-Suffix. So I hope when we will touch this in the future this will be more explicit and we know that in this case the categories come from the categories thesaurus.

@Angi-Kinas
Copy link
Contributor Author

@tkohr Yes there are only these three values because others were not used in the records yet (afaik).

@Angi-Kinas Angi-Kinas merged commit 4e8021b into main May 23, 2024
7 checks passed
@Angi-Kinas Angi-Kinas deleted the change-keywords-to-categories-dataset-page branch May 23, 2024 11:42
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.

2 participants