Skip to content

Conversation

cocoyeal
Copy link
Contributor

Note: in this PR, i only fixed the broken links regarding dir tooling. but there are more broken links regarding other dir like nodes and reference. i will draft other PRs to fix them one by one since the number of files being affected will be big.

Why this should be merged

broken links being fixed.

How this works

by replacing the broken links with the right links.

How this was tested

by manually checking if the replaced links works.

@cocoyeal cocoyeal requested a review from StephenButtolph as a code owner May 19, 2024 04:09
Copy link
Contributor

@StephenButtolph StephenButtolph left a comment

Choose a reason for hiding this comment

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

The service.md files are actually used to generate the https://docs.avax.network/ website. These links are intentionally set to local paths for the website.

I checked the website and it seems like these links are working as expected there. If you find any broken links on the site please make a new PR to fix those links here!

enabled, use [config flag `--api-admin-enabled=true`](/nodes/configure/avalanchego-config-flags.md#--api-admin-enabled-boolean).

This API set is for a specific node, it is unavailable on the [public server](/tooling/rpc-providers.md).
This API set is for a specific node, it is unavailable on the [public server](https://docs.avax.network/tooling/rpc-providers).
Copy link
Contributor

Choose a reason for hiding this comment

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

For example, this page is rendered here: https://docs.avax.network/reference/avalanchego/admin-api and the link seems to work as expected.

@cocoyeal
Copy link
Contributor Author

Hi @StephenButtolph , thanks for the info. i do find one broken link, will make a new PR to fix it.

@cocoyeal cocoyeal mentioned this pull request May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants