feat: add dex-general and dex-stable rpc defs#103
Conversation
Signed-off-by: Gregory Hill <gregorydhill@outlook.com>
| "params": [ | ||
| { | ||
| "name": "supply", | ||
| "type": "AssetBalance" |
There was a problem hiding this comment.
Just tested and I think not, although I'm not sure if I'm missing something or if this was indeed fixed in serde
There was a problem hiding this comment.
Btw many of the rpcs using FixedU128 would have the same issue
There was a problem hiding this comment.
Btw many of the rpcs using
FixedU128would have the same issue
That's not true, FixedU128 has a customSerialize implementation that serializes as string (similarly to our balance wrapper)
There was a problem hiding this comment.
But if you tested then I guess they must have fixed it upstream. Edit: it seems you just need the arbitrary_precision feature. We don't explicitly enable it but maybe one of our dependencies does serde-rs/json#548 . Edit 2: cargo tree -e features does not show the arbitrary_precision feature being enabled..
2 participants
Signed-off-by: Gregory Hill gregorydhill@outlook.com