Skip to content

fix: tax calculation#777

Merged
leoslr merged 3 commits intomasterfrom
fix-tax-calculation
Feb 16, 2022
Merged

fix: tax calculation#777
leoslr merged 3 commits intomasterfrom
fix-tax-calculation

Conversation

@leoslr
Copy link
Contributor

@leoslr leoslr commented Feb 16, 2022

BigNumber.from(Number(x)) introduced an issue for large input.
I added a new test case to prevent from this error in the future

@leoslr leoslr changed the title Fix tax calculation fix: tax calculation Feb 16, 2022
@coveralls
Copy link

Coverage Status

Coverage remained the same at 89.469% when pulling 6227c00 on fix-tax-calculation into d1281e6 on master.

@leoslr leoslr enabled auto-merge (squash) February 16, 2022 16:17
@leoslr leoslr merged commit 629b908 into master Feb 16, 2022
@leoslr leoslr deleted the fix-tax-calculation branch February 16, 2022 16:24
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