-
Notifications
You must be signed in to change notification settings - Fork 193
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
Challenge 1 extension #238
Challenge 1 extension #238
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.
Lgtm! Added some small changes you need to do
extension/README.md.args.mjs
Outdated
> in a second terminal window, 🛰 deploy your contract (locally): | ||
|
||
\`\`\`sh | ||
cd challenge-1-decentralized-staking |
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.
Remove this line
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.
Removed
extension/README.md.args.mjs
Outdated
> in a third terminal window, start your 📱 frontend: | ||
|
||
\`\`\`sh | ||
cd challenge-1-decentralized-staking |
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.
Remove this line
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.
Removed
</div> | ||
`; | ||
|
||
export const externalExtensionName = "SpeedRunEthereum Challenge #1"; |
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.
please add empty line at the end of this file and other files when you see that minus sign. If you use vscode/cursor add this to your settings "files.insertFinalNewline": true
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.
Added empty line and added the config to settings. Thanks!
Thanks!! Changes done! |
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.
Nice job Damu! Added a couple of comments I found when tried to yarn vercel
.
extension/packages/nextjs/app/staker-ui/_components/EthToPrice.tsx
Outdated
Show resolved
Hide resolved
Thanks!! Changes done. Anyway, if you try to deploy to Vercel you have to resolve the challenge first to get no errors. |
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.
Thanks! Lgtm!
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.
LGTM! Merging 🙏
Challenge 1 migrated to an extension following #234
Closes #235