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

Revamps build/index #235

Closed
wants to merge 3 commits into from
Closed

Revamps build/index #235

wants to merge 3 commits into from

Conversation

dawnkelly09
Copy link
Collaborator

Description

Please explain the changes this PR addresses here.

  • adds copy to the build/index page to guide the user journey
  • renames the rendered title to Build Multichain
  • adds links for pending Product Comparison Guide and Use Cases pages/sections
  • removes the navigation cards
  • renames "Build Frontend Applications" nav heading/section name to "Application Toolkit"
  • renames "Build Contract Integrations" nav heading/section name to "Transfer Protocols"
    Note: I did not update file/directory names yet until we know if/how much of this we want to implement

Things remaining to address (not changed with this PR):

  • I would like to remove the current Toolkit section based on:
    • Wormhole scan entry is just a link, not even a page
    • Wormhole SDK nav entry is duplicated and links to the same pages in the Application Toolkit section
    • The Solidity SDK page makes more sense being in Transfer Protocols where the smart contract guides live
    • The FAQs here would be better served moving to the section to which they pertain and/or adding to existing FAQ page
    • Tilt page should move to Start Building section as it dev env setup materials
    • Move Wormhole CLI to Start Building section as it is for all types of WH builders to use
    • Wormhole Spy SDK link on card can move to Related Resources on Spy page
    • VAA Parser card can be a Related Resource on VAA page

Checklist

  • [x ] Required - I have added a label to this PR 🏷️
  • [x ] Required - I have run my changes through Grammarly
  • If pages have been moved, I have created redirects in the wormhole-mkdocs repo - I didn't move anything. Yet!

@dawnkelly09 dawnkelly09 added A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) B0 - Needs Review Pull request is ready for review labels Feb 6, 2025
@dawnkelly09 dawnkelly09 requested a review from eshaben as a code owner February 6, 2025 17:32
@dawnkelly09 dawnkelly09 removed the B0 - Needs Review Pull request is ready for review label Feb 6, 2025
@dawnkelly09 dawnkelly09 marked this pull request as draft February 6, 2025 18:27
@dawnkelly09
Copy link
Collaborator Author

Closing as these items are covered in the new IA PRs

@dawnkelly09 dawnkelly09 deleted the dawn/product-experiment-docs branch March 12, 2025 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant