-
Notifications
You must be signed in to change notification settings - Fork 929
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
warn the user about evm version #3629
Conversation
✅ Deploy Preview for remixproject ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@@ -5,6 +5,7 @@ | |||
"udapp.value": "Value", | |||
"udapp.contract": "Contract", | |||
"udapp.compiledBy": "Compiled by {compilerName}", | |||
"udapp.warningEvmVersion": "Please make sure that the current network is following the evm version: {evmVersion}. Otherwise any deployment will fail.", |
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 think you don't need "that" here
a6a5151
to
b354e07
Compare
b354e07
to
d7df1a5
Compare
|
and I don't know why the mouse pointer is not working on hover for only this badge. |
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.
see comments.
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.
yes, but one need to make sure the right evm version is selected in "run and deploy"
Compilation can also come from other sources (hardhat, ..) |
84bf377
to
458bcad
Compare
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
458bcad
to
984be73
Compare
984be73
to
0616614
Compare
fix #3628