Change API links to full urls #802
Conversation
| @@ -19,7 +19,7 @@ First, enable EdgeDB in your project’s settings. | |||
|
|
|||
| ![Project Settings]Now, let's save some data to EdgeDB. | |||
|
|
|||
| To do this, we’ll be following the EdgeDB docs from the [Bucketing API](/bucketing-api/#tag/EdgeDB). | |||
| To do this, we’ll be following the EdgeDB docs from the [Bucketing API](https://docs.devcycle.com/bucketing-api/#tag/EdgeDB). | |||
There was a problem hiding this comment.
Won't this break testing locally?
There was a problem hiding this comment.
Yea it does but I'm not sure how to fix these otherwise. Should we hold off?
There was a problem hiding this comment.
Another way to go about these broken anchors is to use HTML. So the above would look like...
To do this, we’ll be following the EdgeDB docs from the <a href="/bucketing-api/#tag/EdgeDB">Bucketing API</a>.
The link works, docusaurus doesn't complain and you can still test locally. However, this looks like it bypasses the broken link detection which is also not great.
There was a problem hiding this comment.
Do these links work locally / prod? like they look correct to me why is the broken link detection even triggering for them?
There was a problem hiding this comment.
Yup, they all work in local and prod. I'm not sure why Docusaurus flags them, but both the API docs and CLI docs are generated by plugins so it's likely related to that.
Changes the Management and Bucketing API URLs to full URLs to resolve broken anchors