Skip to content

fix(savanna): resolve Create API key links through Antora xrefs - #177

Merged
Tushar-TG-14 merged 1 commit into
tigergraph:mainfrom
ngarakapati:fix/create-api-key-xref
Sep 28, 2026
Merged

Tushar-TG-14 merged 1 commit into
tigergraph:mainfrom
ngarakapati:fix/create-api-key-xref

Conversation

@ngarakapati

Copy link
Copy Markdown
Contributor

Summary

  • Endpoint Authorizations links were hand-written ../create-api-key.html anchors inside a passthrough block. PR fix(savanna): keep Create API key link on endpoint pages #176 only added xref CSS classes, so Antora never resolved the URL.
  • Production rewrites leftover .html hrefs to site-root paths (/savanna/...), which drop /docs and 404 on www.tigergraph.com.
  • The generator now emits a real AsciiDoc xref. A production-style build publishes href="../create-api-key" on all 54 API-key endpoint pages, matching the overview page.

Test plan

  • Rebuild the docs site and open an endpoint page such as Check the current IP.
  • Click Create an API key under Authorizations and confirm it opens /docs/savanna/main/rest-api/create-api-key.
  • Confirm the same link on the REST API overview still works.

Hand-written .html hrefs were rewritten to site-root paths that drop /docs and 404.
@netlify

netlify Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for thriving-strudel-91d4a9 ready!

Name Link
🔨 Latest commit 5e5df08
🔍 Latest deploy log https://app.netlify.com/projects/thriving-strudel-91d4a9/deploys/6ab945e693994c000858fde7
😎 Deploy Preview https://deploy-preview-177--thriving-strudel-91d4a9.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Tushar-TG-14
Tushar-TG-14 merged commit 65795a2 into tigergraph:main Sep 28, 2026
4 of 5 checks passed
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