Skip to content
This repository has been archived by the owner on Jan 18, 2024. It is now read-only.

Ensure EOSJS supports BigInts #25

Open
Tracked by #39
ericpassmore opened this issue Dec 22, 2022 · 0 comments
Open
Tracked by #39

Ensure EOSJS supports BigInts #25

ericpassmore opened this issue Dec 22, 2022 · 0 comments

Comments

@ericpassmore
Copy link
Contributor

ericpassmore commented Dec 22, 2022

Review code for numbers passed as parameters. Look to support BigInts. From issue originally reported in AntelopeIO/leap#579

  • Review all functions calls with numeric params in EOSJS for BigInt support.
    • safest thing is to make all numbers BigInts, as numeric operations my result in BigInt
  • Consider specialized functions to wrap BigInt -> Numeric conversion
    • safely convert to unsigned
    • customized exception when number can not be converted
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant