aptos-labs / developer-docs

Source for the Aptos developer docs
https://aptos.dev
Apache License 2.0
854 stars 84 forks source link

Fix the organization of Nextra #335

Closed jmintuitive closed 4 months ago

jmintuitive commented 4 months ago

Description

Moved the pages into the New Docs Format.

Wrote a script to automatically fix links called update-relative-links.py. Used it in conjunction with markdown-link-checker to verify that once moved all the relative links worked. (Still some external links which need to be fixed, and a couple of pages which need to be migrated - see below).

Remaining issues to be resolved in future PRs:

Checklist

Pages that still need migrating (Overall, not in this PR - but they were broken links)

Remaining TODOs on migration effort

vercel[bot] commented 4 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
developer-docs-nextra ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 17, 2024 0:14am
netlify[bot] commented 4 months ago

Deploy Preview for aptos-developer-docs ready!

Name Link
Latest commit 79fad7d5639e82a65f6c9961df1e87f4f2e6c9f2
Latest deploy log https://app.netlify.com/sites/aptos-developer-docs/deploys/6646a0ed4d382700086988e3
Deploy Preview https://deploy-preview-335--aptos-developer-docs.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 site configuration.

jmintuitive commented 4 months ago

2. We need to find another place to put the Wallet markdown doc in Standards

Why do we need to find another place to put the Wallet markdown doc?

hariria commented 4 months ago

2. We need to find another place to put the Wallet markdown doc in Standards

Why do we need to find another place to put the Wallet markdown doc?

The Wallet standards are unrelated to Smart Contracts

jmintuitive commented 4 months ago

nit: I noticed when merging that we're adding → to every Card.Title. That seems really easy to forget over time, plus a bit annoying to find that character to copy + paste when adding new cards - @hariria would it make sense to add that to the Card.Title component instead? (With an optional flag to turn off potentially).