Skip to content
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

reference docs migration #1702

Merged
merged 21 commits into from
Mar 25, 2024
Merged

Conversation

meaghanfitzgerald
Copy link
Collaborator

@meaghanfitzgerald meaghanfitzgerald commented Mar 11, 2024

Why this should be merged

puts api docs (like this one) directly in https://github.com/ava-labs/avalanchego

How this works

I removed all the AvalancheGo API docs from this repository and moved the content to each API's file in https://github.com/ava-labs/avalanchego. A docusaurus plugin I've implemented in the past is fetching the raw contents and importing them to our site during the build. Details on how I implemented this can be found here.

For example, the metrics-api.md was moved from
https://github.com/ava-labs/avalanche-docs/blob/master/docs/reference/avalanchego/metrics-api.md
to
https://github.com/ava-labs/avalanchego/blob/meag/docs-format/api/metrics/service.md.

Note: once @StephenButtolph merges my changes, this second link will become https://github.com/ava-labs/avalanchego/blob/master/indexer/service.md .

How these changes were tested

  • yarn build
  • in conjunction with changes made in this branch of avalanchego

Copy link

cloudflare-workers-and-pages bot commented Mar 11, 2024

Deploying avalanche-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 41ae805
Status: ✅  Deploy successful!
Preview URL: https://19cd61e7.avalanche-docs.pages.dev
Branch Preview URL: https://meag-reference-docs-migratio.avalanche-docs.pages.dev

View logs

@meaghanfitzgerald meaghanfitzgerald changed the title Meag/reference docs migration reference docs migration Mar 12, 2024
@meaghanfitzgerald meaghanfitzgerald marked this pull request as ready for review March 12, 2024 23:18
@meaghanfitzgerald meaghanfitzgerald added the DO NOT MERGE This PR is not meant to be merged in its current state label Mar 12, 2024
@meaghanfitzgerald
Copy link
Collaborator Author

blocked by ava-labs/avalanchego#2845

@meaghanfitzgerald meaghanfitzgerald merged commit 2f245e5 into master Mar 25, 2024
7 of 8 checks passed
@meaghanfitzgerald meaghanfitzgerald deleted the meag/reference-docs-migration branch March 25, 2024 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE This PR is not meant to be merged in its current state
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant