-
-
Notifications
You must be signed in to change notification settings - Fork 229
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
Country name site search improvements #2729
Comments
/cc @ikesau |
Due to the highly templated nature of country profiles, we may want to see if there's a different way to think about surfacing them in search. i.e. it won't be helpful to have 180 results when you type "Energy", but maybe if you type a recognizable country name, we could add a custom section for country profiles. I've started a discussion on Slack to see how others are feeling about this. |
Just to give a fuller picture, I did an analysis of all the things that are in our sitemap but missing from Algolia: https://gist.github.com/larsyencken/1fcd4e026a4b67b157263f2ae1366a1d The list seems to contain:
|
Haven't checked these, but if a chart is fully untagged, or only tagged with "private" tags like core-econ, then it will not get indexed to Algolia. It probably also shouldn't go into the sitemap then, but it probably get included anyways. |
Thanks for that, Lars! I think of the classes of omissions that you listed, country profiles are the only ones we actually want to handle, and they probably require some sort of special treatment. I'm going to update this ticket to reflect that. |
This issue has had no activity within 10 months. It is considered stale and will be closed in 7 days unless it is worked on or tagged as pinned. |
This is done now, see #3483 |
We aren't indexing country profiles (currently for CO2, Covid, and Energy) in Algolia, so users have no way of finding them.
This probably requires a new section in our search, so that typing "Energy" doesn't make half the results country profiles for random countries.
We also should make it so that if you type a country name, Grapher charts that return are preconfigured with that country name as their selected entity (we could use CF Workers to generate the preview thumbnail now 🙌)/ ✅ Done in #3483The text was updated successfully, but these errors were encountered: