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

Fix navigation and formatting in Build -> Smart Contracts (Move) -> Objects #680

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

igor-p
Copy link
Collaborator

@igor-p igor-p commented Oct 25, 2024

Description

On https://aptos.dev/en/build/smart-contracts/objects/creating-objects, the sidebar table of contents is broken (due to the wrong headings used in the mdx file). This fixes the ToC and ensures correct headings are used.

Another issue is that routing to the Objects page under the Addressable Storage section is broken. On clicking it, the page will navigate correctly, but the sidebar shows you in Object under Aptos Standards (they both use objects as the route). This fixes that issue by using object path for the route under "Aptos Standards" and move-objects for the route under "Addressable Storage".

I added redirects for the old routes.

Screen.Recording.2024-10-25.at.2.21.06.PM.mov

Checklist

  • Do all Lints pass?
    • Have you ran pnpm spellcheck?
    • Have you ran pnpm fmt?
    • Have you ran pnpm lint?

@igor-p igor-p requested review from a team, gregnazario and hariria as code owners October 25, 2024 18:23
Copy link

vercel bot commented Oct 25, 2024

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 Oct 25, 2024 8:03pm

Comment on lines 26 to 27
objects: {
title: "Object",
"aptos-objects": {
title: "Objects",
Copy link
Collaborator Author

@igor-p igor-p Oct 25, 2024

Choose a reason for hiding this comment

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

I tried keeping this route as objects, but if you go to https://aptos.dev/en/build/smart-contracts/objects/creating-objects and hit refresh, it redirects to using-objects and I'm not sure why. Changing the name of the route to something other than objects fixes it though.

@igor-p igor-p marked this pull request as draft October 25, 2024 18:42
@igor-p igor-p force-pushed the igor/fix-build-SC-objects-pages branch from db28fab to 197e84e Compare October 25, 2024 19:52
@igor-p igor-p marked this pull request as ready for review October 25, 2024 19:53
@igor-p igor-p requested a review from a team as a code owner October 25, 2024 19:53
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.

1 participant