-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat(devx) - Navigation Changes #4048
Conversation
… visibility, and give the users a better overview of the developer section
✅ Vercel Preview Deployment is ready! |
✅ Vercel Preview Deployment is ready! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes are nice, but I have those two comments:
1- I think access on-chain time should be moved to Move Overview, it looks a bit "lonely" on the developer landing page
2- Subscribing to events on chain should belong to SDKs and CLI somehow, WDYT?
✅ Vercel Preview Deployment is ready! |
I agree, nice changes. Lgtm |
…anges # Conflicts: # docs/content/sidebars/references.js
✅ Vercel Preview Deployment is ready! |
Description of change
Important Notice
I did not yet move the files to match the restructure, only altered the sidebar. If this new navigation is an improvement, we should update the files and folders to match.
Links to any relevant issues
Be sure to reference any related issues by adding
fixes #(issue)
.Type of change
Choose a type of change, and delete any options that are not relevant.
How the change has been tested
Describe the tests that you ran to verify your changes.
Make sure to provide instructions for the maintainer as well as any relevant configurations.
Change checklist
Tick the boxes that are relevant to your changes, and delete any items that are not.
Release notes
Updated docs nav