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

feat(TxBuilder): Integrate amount formatter to transaction builder #897

Merged
merged 13 commits into from
Feb 19, 2020

Conversation

nduchak
Copy link
Contributor

@nduchak nduchak commented Feb 12, 2020

Add toAettos to amount-formatter
Add denomination options to each high lvl API which use TxBuilder(default: 'aettos')
Adjust bignumber helper documentation
Add test's

Add `toAettos` to `amount-formatter`
Add `denomination` options to each high lvl API which use TxBuilder(default: 'aettos')
Adjust `bignumber` helper documentation
Add test's
@nduchak nduchak added this to the 7.1.0 milestone Feb 12, 2020
@nduchak nduchak requested a review from mradkov February 12, 2020 16:49
@nduchak nduchak self-assigned this Feb 12, 2020
@nduchak
Copy link
Contributor Author

nduchak commented Feb 12, 2020

closes #382

@codecov
Copy link

codecov bot commented Feb 12, 2020

Codecov Report

Merging #897 into develop will increase coverage by 12.69%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           develop     #897       +/-   ##
============================================
+ Coverage    79.97%   92.67%   +12.69%     
============================================
  Files           57       57               
  Lines         2902     2923       +21     
============================================
+ Hits          2321     2709      +388     
+ Misses         581      214      -367     
Impacted Files Coverage Δ
es/utils/json-big.js 81.25% <0.00%> (+0.48%) ⬆️
es/contract/compiler.js 100.00% <0.00%> (+2.08%) ⬆️
es/utils/crypto.js 85.47% <0.00%> (+5.12%) ⬆️
es/tx/tx.js 100.00% <0.00%> (+14.45%) ⬆️
es/channel/internal.js 95.31% <0.00%> (+77.34%) ⬆️
es/channel/handlers.js 79.27% <0.00%> (+77.47%) ⬆️
es/channel/index.js 88.54% <0.00%> (+80.20%) ⬆️

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 c5fa163...f1fdabc. Read the comment docs.

@nduchak nduchak force-pushed the feat/amount-formatter-for-tx-builder branch from e613848 to fd31fbd Compare February 13, 2020 14:43
@nduchak nduchak force-pushed the feat/amount-formatter-for-tx-builder branch from b092f94 to 75bc3fc Compare February 13, 2020 15:06
@nduchak nduchak requested a review from mradkov February 17, 2020 10:46
@nduchak nduchak merged commit a9f4934 into develop Feb 19, 2020
@nduchak nduchak deleted the feat/amount-formatter-for-tx-builder branch February 20, 2020 12:02
@nduchak nduchak mentioned this pull request Feb 25, 2020
@mradkov mradkov mentioned this pull request Jun 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants