-
Notifications
You must be signed in to change notification settings - Fork 40
fix(pop up contract): signer prompted if not provided #725
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
Conversation
Codecov Report❌ Patch coverage is @@ Coverage Diff @@
## main #725 +/- ##
=======================================
Coverage 77.05% 77.06%
=======================================
Files 111 111
Lines 25422 25471 +49
Branches 25422 25471 +49
=======================================
+ Hits 19589 19629 +40
- Misses 3772 3773 +1
- Partials 2061 2069 +8
🚀 New features to boost your workflow:
|
Codecov Report❌ Patch coverage is @@ Coverage Diff @@
## main #725 +/- ##
==========================================
- Coverage 76.28% 76.27% -0.01%
==========================================
Files 115 115
Lines 25909 26016 +107
Branches 25909 26016 +107
==========================================
+ Hits 19765 19845 +80
- Misses 4015 4029 +14
- Partials 2129 2142 +13
🚀 New features to boost your workflow:
|
* refactor: pop up ink node automatic deployment * fix: consistent interaction with contract after deployment for suri or wallet
e0a35b6 to
d1df96c
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 now! We still have to decide what to do if --skip-confirm is on and the user needs to specify parameters for deploying or calling contracts. Still, I think we can leave it as-is for now.
|
Can we also check whether with the --skip-confirm flag the user is not prompted for confirmation on executing the message @moliholy |
That's the case. No confirmation is requested when deploying or executing if |
When deploying a contract the signer was no prompted if not provided and alice was always used. This fixes that.
Closes #721
Closes #732