-
Notifications
You must be signed in to change notification settings - Fork 5
feat: Add Knowledge Base Content #86
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
base: dev
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
0xOneTony
left a comment
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 new section looks good, left a comment about restructuring a bit. Lets also fix the CI
|
|
||
| The onboarding is divided into two key sections: | ||
|
|
||
| - [Knowledge Base](/docs/development/solidity/onboarding/knowledge-base): A comprehensive guide covering coding standards, best practices, and essential tools. |
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.
I think it would be better to keep the knowledge base section as 1 page like it was and add a new section under Coding Standards that will have a small description and link to a new page to the side bar on the solidity sections like the "Coding Style" and "Natspec" one.
I see that the UI onboarding is also using the Defi and Dapps section, so doing what I describe above would benefit them also with just linking to the new page we create https://handbook.wonderland.xyz/docs/development/frontend/onboarding/front-knowledge-base#defi--dapps
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.
Updated here bd76e02!
|
|
||
| ### Core Infrastructure | ||
| - [Chainlink](https://docs.chain.link/data-feeds) - The global standard for blockchain connectivity. | ||
| - [Connext](https://docs.connext.network/) - Cross-chain communication protocol. |
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.
Connext changed to Everclear we should update this, maybe remove it? or move it to a new section DeFi ?
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.
That's curious. I checked the docs website before maintaining it, and there wasn't a warning about it. Now, I found it.
Let's move it to DeFi.
Description
This PR enhances the Solidity onboarding module to include the Knowledge Base session available on Linear. It also removes outdated content and contains new core content aligned with Wonderland goals.
Specifications
Convert the Knowledge Base page into a new sub-session (dropdown menu) to enable future additions related to the topic without any new structural change. It also updates all links that were pointing to the old page.
Considerations
Recommended Resourcestitle.