-
Notifications
You must be signed in to change notification settings - Fork 354
3966 update glossary #4164
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
base: main
Are you sure you want to change the base?
3966 update glossary #4164
Conversation
…into 3966-update-glossary
…ng script for re-use. Starting with concepts and getting started to reduce scope.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Skipped Deployments
|
Wow! This looks super cool! 👏 I've just been looking through and realised there is one challenge to this I hadn't considered before, which is that when we run the translations the LLM is not going to know what to do with this component in the markdown. It will translate the text around it and we will be left with the translation and glossary words still in English, so by introducing this component around the words we're "polluting" the markdown a little bit. I also wonder how this affects LLMs as they parse the markdown? One potential solution is that we insert these components only when we build the docs. We should be able to do this using the Docusaurus life cycle hooks but then processing becomes trickier. What if we introduced some custom markdown syntax like Thoughts? |
@Blargian - You bring up some really good points. I think this is something that we need to address, I'll look into transforming custom markdown syntax, great idea. |
…into 3966-update-glossary
…into 3966-update-glossary
…o a GlossaryToolTip at build time.
@Blargian - Updates have been made for the new approach for tool tips, along with added steps to extract the current glossary terms and check if terms are appropriately wrapped in the docs. This will only surface a warning and will not fail the build. You'll see something like this on success. ![]() |
Summary
Adding custom tooltips that have been injected via script, based off of a json file generated off of the current glossary page. This was executed on concepts and getting-started pages only as a first pass to reduce scope.
Checklist
ClickHouse.Docs.Glossary.Tooltip.mp4