Skip to content

Conversation

@torresmateo
Copy link
Collaborator

This adds the toolkit name to each tool heading so it's easy to avoid naming issues

@vercel
Copy link

vercel bot commented Jun 18, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 18, 2025 6:44pm

@torresmateo torresmateo requested a review from byrro June 18, 2025 17:29
Copy link
Member

@byrro byrro left a comment

Choose a reason for hiding this comment

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

There's one thing we haven't anticipated: the table of contents in "Available Tools" also needs to be updated with the new tool names, otherwise clicking it doesn't lead to the corresponding tool heading.

@torresmateo torresmateo requested a review from byrro June 18, 2025 18:40
@torresmateo
Copy link
Collaborator Author

I had to modify the component to account for the dot dividing the toolkit name and tool names, but the links are now restored

Copy link
Member

@byrro byrro left a comment

Choose a reason for hiding this comment

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

LGTM

@torresmateo torresmateo marked this pull request as ready for review June 18, 2025 20:11
@torresmateo torresmateo merged commit fb8edd9 into main Jun 18, 2025
4 checks passed
@torresmateo torresmateo deleted the mateo/tools-titles branch June 18, 2025 20:11
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.

3 participants