-
Notifications
You must be signed in to change notification settings - Fork 40
[Weave] Corrects syntax for source link URLs in Weave ref docs #2039
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
Conversation
lazydocs generates <img ... > (no self-closing slash) but the regex expected <img ... />. Updated pattern to handle both formats with /?>
- Create compact SourceLink component matching GitHubLink styling - Add CSS for .source-link class (compact button style) - Update generation script to use <SourceLink url=... /> component
📚 Mintlify Preview Links✨ Added (1 total)📄 Pages (1)
📝 Changed (1 total)⚙️ Other (1)
🤖 Generated automatically when Mintlify deployment succeeds |
🔗 Link Checker Results✅ All links are valid! No broken links were detected in the changed files. Tip Redirects detected: If you see redirects for internal docs.wandb.ai links, check if they have trailing slashes. Mintlify automatically removes trailing slashes, causing redirects like:
Fix: Remove trailing slashes from links to avoid unnecessary redirects. |
Description
Currently, the links to the source code link that send readers to the code in the Weave SDK are 404ing. This hopefully corrects that. It also make the source links a button instead of an image that does a weird zoom animation.