-
-
Notifications
You must be signed in to change notification settings - Fork 805
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
Added Web3 Staking Dapp #767
Conversation
@nirban256 is attempting to deploy a commit to a Personal Account owned by @reactplay on Vercel. @reactplay first needs to authorize it. |
@atapas pls review the pr |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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 have given some initial comments.
@Sachin-chaurasiya will do deep review.
src/plays/web3-staking-dapp/artifacts/build-info/5faf9c6c2f8753632f3c7c13937cd196.json
Show resolved
Hide resolved
src/plays/web3-staking-dapp/artifacts/contracts/Staking.sol/Staking.dbg.json
Show resolved
Hide resolved
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.
Some initial comments
src/plays/web3-staking-dapp/smart-contract/cache/solidity-files-cache.json
Outdated
Show resolved
Hide resolved
If all the conditions are met then please merge the pr |
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.
@nirban256 Few more comments and remove the src/plays/web3-staking-dapp/styles.css
as it's empty.
src/plays/web3-staking-dapp/smart-contract/cache/solidity-files-cache.json
Outdated
Show resolved
Hide resolved
@nirban256 , Build is failing, As we have deleted the styles.css and we have to remove the reference of it as well. |
removed the line @Sachin-chaurasiya |
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.
Great work @nirban256
thanks @Sachin-chaurasiya |
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.
Approving
Description
It is a Staking Dapp where people will get interest based on the amount they stake after a specific period of time. I have added some dependencies namely hardhat and ethers in order to build the Dapp (Decentralized Application).
Fixes #550
Type of change
How Has This Been Tested?
Checklist: