Skip to content

Handle token's decimals property#86

Merged
samuelmanzanera merged 2 commits intomasterfrom
Add-decimals
Sep 30, 2022
Merged

Handle token's decimals property#86
samuelmanzanera merged 2 commits intomasterfrom
Add-decimals

Conversation

@Neylix
Copy link
Copy Markdown
Member

@Neylix Neylix commented Sep 30, 2022

Fixes #85

Add a new network function to get the definition of a token.
Add new parameter to fromBigInt and toBigInt function which permit to define the number of decimals needed

Add unit test on fromBigInt and toBigInt function.

getTokenfunction tested with transactionBuilder example

@samuelmanzanera
Copy link
Copy Markdown
Member

Seems good.
Could you just write also tests for getToken API call.

@Neylix
Copy link
Copy Markdown
Member Author

Neylix commented Sep 30, 2022

Seems good. Could you just write also tests for getToken API call.

done !

@samuelmanzanera samuelmanzanera merged commit 45159e5 into master Sep 30, 2022
@samuelmanzanera samuelmanzanera deleted the Add-decimals branch September 30, 2022 13:16
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.

Update fromBigInt and toBigInt function with decimal part

2 participants