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

Fixed sentence structure & spelling errors #7

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Fixed sentence structure & spelling errors #7

wants to merge 15 commits into from

Conversation

susmakhadka
Copy link

#susma7077

haardikk21 and others added 15 commits December 16, 2021 12:39
* Update mood.sol contract to same solidity version used in the README.md

I think it is confusing if the README.me is using the 'string memory' but the full example contract dosn't.

* Link mood.sql full example to https://github.com/LearnWeb3DAO

The mood.sol full example was linked to https://github.com/BlockDevsUnited, so it could not be updated.

Co-authored-by: Philipp Dahse <[email protected]>
I've added a line about clicking Ethereum Mainnet button to switch to Ropsten Testnet. Also I've added another faucet for Ropsten Network because of some technical issues happening with the current one.

Co-authored-by: Sneh Koul <[email protected]>
* Spelling/Phrasing Fix, Missing Links

* Update README.md

Resolved initial PR issues.
Discord #susma7077
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants