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

🎉 Link from admin indicator search to ETL semantic search #4569

Merged
merged 2 commits into from
Feb 17, 2025

Conversation

larsyencken
Copy link
Contributor

@larsyencken larsyencken commented Feb 17, 2025

We have a great semantic search over indicators in the ETL, but people may not be aware of it or have used it.

This change adds a link from the existing indicator search, making it easy to discover.

@owidbot
Copy link
Contributor

owidbot commented Feb 17, 2025

Quick links (staging server):

Site Dev Site Preview Admin Wizard Docs

Login: ssh owid@staging-site-link-to-semantic-search

SVG tester:

Number of differences (default views): 0 ✅
Number of differences (all views): 0 ✅

Edited: 2025-02-17 16:28:57 UTC
Execution time: 1.17 seconds

@larsyencken larsyencken requested a review from danyx23 February 17, 2025 16:29
@larsyencken larsyencken merged commit 80d940a into master Feb 17, 2025
15 checks passed
@larsyencken larsyencken deleted the link-to-semantic-search branch February 17, 2025 20:54
@larsyencken
Copy link
Contributor Author

Since it's low risk, I'll just merge it.

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