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.

Sign up for GitHub

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

Replace toWei/fromWei when base token conversion #1318

Merged
merged 21 commits into from
Mar 16, 2022
Merged

Replace toWei/fromWei when base token conversion #1318

Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
a70b3f5
replaced toWei inside FRE
KatunaNorbert Feb 25, 2022
7f3f04d
added amountToUnits for maxPrice inside pool
KatunaNorbert Feb 28, 2022
21beab9
replaced fromWei
KatunaNorbert Mar 1, 2022
ea5999a
add web3 param to getPoolCreationParams
KatunaNorbert Mar 2, 2022
5ec5c06
added await before amountToUnits inside swapExactAmount
KatunaNorbert Mar 2, 2022
2ece3c7
swapExactAmount update
KatunaNorbert Mar 2, 2022
f6bcd73
add await to unitsToAmount inside setRate
KatunaNorbert Mar 2, 2022
3322aca
convert newRate to string
KatunaNorbert Mar 2, 2022
5da7242
replaced unitToAmounts with amountToUnits inside setRate
KatunaNorbert Mar 3, 2022
ca6193b
replace unitToAmounts with amountToUnits
KatunaNorbert Mar 3, 2022
b40d9f2
added await
KatunaNorbert Mar 3, 2022
81a3d3b
replaced String() with toSting()
KatunaNorbert Mar 4, 2022
31ee3f9
replaced String() in Provider
KatunaNorbert Mar 4, 2022
81dd759
transform newRate into wei inside estimateSetRate
KatunaNorbert Mar 4, 2022
433577a
replace toWei inside getFreCreationParams
KatunaNorbert Mar 4, 2022
cbba765
added web3 param to getFreCreationParams
KatunaNorbert Mar 4, 2022
bdc60a5
add amountToUnits inside getFreOrderParams
KatunaNorbert Mar 4, 2022
62cc124
added logs
KatunaNorbert Mar 7, 2022
8dd8f34
removed unitsToAmount from fixed rate
KatunaNorbert Mar 8, 2022
a62666d
removed amountToUnits inside marketFee
KatunaNorbert Mar 8, 2022
cee1bc2
removed logs
KatunaNorbert Mar 8, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions src/pools/fixedRate/FixedRateExchange.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -366,10 +366,10 @@ export class FixedRateExchange {
const estGas = await this.estSetRate(
address,
exchangeId,
await this.unitsToAmount(exchange.baseToken, String(newRate))
await this.amountToUnits(exchange.baseToken, String(newRate))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why noy toString()?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is what was used above so I used to be consistent

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

strange, we always use toString, don't know how that stayed there.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will change it

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

feel free to update all String(value) in other places if you find more of this 😃

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, I will

)
const trxReceipt = await this.contract.methods
.setRate(exchangeId, await this.unitsToAmount(exchange.baseToken, String(newRate)))
.setRate(exchangeId, await this.amountToUnits(exchange.baseToken, String(newRate)))
.send({
from: address,
gas: estGas + 1,
Expand Down