-
Notifications
You must be signed in to change notification settings - Fork 432
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
Backport #1379 #1401 #1418 into v3.x.x
and fix tests
#1439
Conversation
… sending payments with contract msg calls (#1379) * first ver.: transfer_in api function implememted but we can't have it in on-chain env * transfer_in moved to test_api * doc + example updated * Update examples/contract-transfer/lib.rs Co-authored-by: Hernando Castano <[email protected]> * use stmt moved to macro * docs and nitty gritties * moved the macro to the test mod * spell fix * next review round suggestions applied * Use four spaces for macro indentation Co-authored-by: Hernando Castano <[email protected]> Co-authored-by: Hernando Castano <[email protected]>
c5a81ad
to
23d61d4
Compare
v3.x.x
and fix testsv3.x.x
and fix tests
CI passes (except ink-waterfall) for the following toolchain:
Making it all green for the latest Rust would require additional efforts (worth devoting a separate PR if needed). ink_waterfall (currently fails), meanwhile, was already broken on the previous release (though for a different reason) |
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!
Following three PRs to backport:
ink_env::pay_with_call!
helper macro for off-chain emulation of sending payments with contract msg calls (Addink_env::pay_with_call!
helper macro for off-chain emulation of sending payments with contract msg calls #1379)pay_with_call
to take multiple arguments (Allowpay_with_call
to take multiple arguments #1401)match
to a separate constants (Moved constants frommatch
to a separate constants #1418)+ tests fixed