Skip to content
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

add xcm instruction RefundSurplus after Transact #2088

Merged
merged 1 commit into from
May 7, 2022
Merged

Conversation

wangjj9219
Copy link
Member

No description provided.

@wangjj9219 wangjj9219 requested review from xlc and syan095 May 6, 2022 11:26
@codecov
Copy link

codecov bot commented May 6, 2022

Codecov Report

Merging #2088 (8b928f2) into master (0871aa7) will decrease coverage by 0.17%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2088      +/-   ##
==========================================
- Coverage   85.96%   85.78%   -0.18%     
==========================================
  Files          99       97       -2     
  Lines       20086    19523     -563     
==========================================
- Hits        17266    16747     -519     
+ Misses       2820     2776      -44     
Impacted Files Coverage Δ
modules/relaychain/src/lib.rs 77.41% <100.00%> (+0.75%) ⬆️
modules/nft/src/lib.rs
modules/nft/src/tests.rs

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0871aa7...8b928f2. Read the comment docs.

@xlc xlc requested a review from zqhxuyuan May 7, 2022 01:14
@zqhxuyuan
Copy link
Contributor

LGTM

@xlc xlc merged commit 42f41c8 into master May 7, 2022
@xlc xlc deleted the refund-surplus branch May 7, 2022 06:28
syan095 pushed a commit that referenced this pull request May 8, 2022
…rrencyIds

* origin/master:
  add xcm instruction RefundSurplus after Transact (#2088)
  rm dex share duplicate code (#2089)
  precompile (#2049)
  Remove honzon global paramter (#2071)
  update orml (#2077)
  Auto publish predeploy contracts (#2076)

# Conflicts:
#	modules/cdp-engine/src/lib.rs
#	modules/cdp-engine/src/tests.rs
#	runtime/acala/src/lib.rs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants