-
Notifications
You must be signed in to change notification settings - Fork 4
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
int inside of a record seems to be returned as string instead of number #77
Comments
are you sure? I think I tested that some time ago and always received |
so you are referring to the current state of this branch: https://github.com/aeternity/aepp-sdk-js/tree/feature/calldata ? and using this branch brings you the result on the right whereas in the past you got the result on the left? (screenshot) |
Please provide minimal reproducible example using the calldata lib public API only. As @marc0olo mentioned this lib works with native BigInts: https://github.com/aeternity/aepp-calldata-js/blob/master/tests/Decoder.js#L307 |
I had a similar issue in sdk tests, it is probably because |
@davidyuk but that's an issue of the SDK then, right? then it's related to this again, isn't it? => aeternity/aepp-sdk-js#1290 |
Nope, sdk doesn't modify BigInts returned from calldata (I planning to keep it as it is), the bug is in |
@thepiwo did you use |
No description provided.
The text was updated successfully, but these errors were encountered: