-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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(website): add index sections to reference and resources #13490
feat(website): add index sections to reference and resources #13490
Conversation
…resentations, Rollup glossary, and whitepaper, Contract documentation , Contract addresses , Protocol constants, RPC configuration
@jscriptcoder can you review this. |
Hi @developerkunal , thanks a lot for your contribution. I think @d1onys1us would be the right person to review this one. |
@developerkunal i don't mean to block this PR, we could very well merge it in and it would be helpful for users, but there is one issue with this approach which is maintainability. as we add more documentation to these sections the documentation maintainers will always need to remember to update these sections. and also the names for the sections such as "resources" "reference" "contract documentation" should be semantic enough to describe what is under that section without needing a description. but i agree this is really useful as an index / table of contents. the larger initiative / correct approach would actually be if we could write a script somehow that will parse all of the files automatically, and automatically generate an index / table of contents dynamically. this is provided out of the box by docusaurus, but not nextra (i don't think). let me know if this issue sounds interesting and if you want to work on doing it with this approach, otherwise i can review this PR and i think we can merge it in and accept the slightly larger maintenance overhead. thanks for your contribution ❤️ even better is if we can work with the nextra team (cc @shuding), and make the feature implementation to nextra, to help out with this issue and then help all other projects with the same issue. |
@d1onys1us thanks for reply i would love to work on create a script to add table of content but at the moment it will not be that fast if we can merge this and when i will be ready to update script we can remove this content.the issue was why i created it like this because it contains a bit of description which is good for SEO and Google Indexing But i surely can work on the new issue. |
@developerkunal understood, we can bring this in for the time being. but if it starts to get stale or become a maintenance overhead we should remove it. but ideally, we can create that script and work with nextra to get the feature in the right way. thanks :) edit: left a few small comments to address otherwise LGTM |
@d1onys1us i have maded the fixes |
@d1onys1us can you get it merged |
Congrats, your important contribution to this open-source project has earned you a GitPOAP! GitPOAP: 2023 Taiko Contributor: Head to gitpoap.io & connect your GitHub account to mint! Learn more about GitPOAPs here. |
This commit adds new sections to the File Description Page for Taiko Protocol resources, including guidelines for contributing to Taiko Protocol on GitHub, official links, presentations, a Taiko glossary, and the official whitepaper. These additions should make it easier for developers and users to navigate and access relevant information about Taiko Protocol.
@jscriptcoder