Skip to content

Add MultiNamedReservableCurrency #730

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

Merged
merged 7 commits into from
Apr 27, 2022

Conversation

stanly-johnson
Copy link
Contributor

resolves #668

  • adds MultiNamedReservableCurrency trait
  • impls MultiNamedReservableCurrency trait in orml-tokens
  • impls NamedReservableCurrency in orml-tokens

@stanly-johnson stanly-johnson changed the title Add NamedReservableCurrency Add MultiNamedReservableCurrency Apr 24, 2022
@codecov-commenter
Copy link

codecov-commenter commented Apr 24, 2022

Codecov Report

Merging #730 (9460de8) into master (7e2f985) will increase coverage by 0.56%.
The diff coverage is 90.30%.

@@            Coverage Diff             @@
##           master     #730      +/-   ##
==========================================
+ Coverage   75.21%   75.78%   +0.56%     
==========================================
  Files          84       84              
  Lines        7643     7942     +299     
==========================================
+ Hits         5749     6019     +270     
- Misses       1894     1923      +29     
Impacted Files Coverage Δ
currencies/src/mock.rs 94.73% <ø> (ø)
tokens/src/mock.rs 31.14% <ø> (ø)
traits/src/currency.rs 0.00% <ø> (ø)
traits/src/lib.rs 0.00% <ø> (ø)
xtokens/src/mock/para.rs 75.75% <ø> (ø)
xtokens/src/mock/para_relative_view.rs 18.91% <ø> (ø)
tokens/src/lib.rs 83.31% <76.98%> (-1.31%) ⬇️
tokens/src/tests.rs 99.85% <100.00%> (+0.02%) ⬆️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@xlc xlc requested review from shaunxw and zqhxuyuan April 24, 2022 21:29
@xlc xlc merged commit 23f7092 into open-web3-stack:master Apr 27, 2022
@stanly-johnson stanly-johnson deleted the named_reservable branch April 28, 2022 04:18
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.

NamedReservableCurrency equivalent for MultiReservableCurrency
5 participants