-
Notifications
You must be signed in to change notification settings - Fork 5k
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
[Docs] Guides & Tutorials improvement #6676
Conversation
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 see that every single quot has been changed to double quot. Like import { Web3PluginBase } from 'web3';
is changed to import { Web3PluginBase } from "web3";
.
There is no right or wrong here. But this is not our coding-style. And if we copy code from our code base it will be single quot always. As if we write double quot in our code, our prettifier configuration will change it to single quot.
So, could you please revert back to single quotes?
Thanks,
@Muhammad-Altabba Thanks for the heads up, I just changed it in the last commit. |
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.
Regarding the Providers Tutorials, please, after splitting them the Prerequisites
section needs to be updated as no need to mention both Ganache and Geth.
Co-authored-by: Muhammad Altabba <[email protected]>
Co-authored-by: Muhammad Altabba <[email protected]>
Description
Hi guys, I made some major changes in the structure of the categories (folders and files), rewrote some texts, fixed some typos, cleaned some code (formatting), added some other code samples, added some things(e.g, wallet, and accounts section) and created the
Getting started
guide, also I removed the glossary in the sidebar and added it to theguides and tutorials
part.Feel free to go to the branch
guides_improvement
and runyarn start
to see in the local environment all the changes.I didn't change the
events subscription
folder at all since I wasn't able to follow the guide/interact with logs.