This replaces the derived FromJSON instance for EntryId with a hand-written instance that is exactly parallel to the existing instance for TradeId, which is also a newtype over Word64. This fixes the failing getUSDAccountLedger test; see #18.
I don't know when or why this test started to fail. It seems odd that the instances for EntryId and TradeId were different. I believe Coinbase's JSON has used strings for everything, including numbers, as long as I've been paying attention.
We can merge this now or wait to see whether there's anything we can do to fix the rest of the tests, whatever you think makes sense.
This replaces the derived
FromJSON
instance forEntryId
with a hand-written instance that is exactly parallel to the existing instance forTradeId
, which is also a newtype overWord64
. This fixes the failinggetUSDAccountLedger
test; see #18.I don't know when or why this test started to fail. It seems odd that the instances for
EntryId
andTradeId
were different. I believe Coinbase's JSON has used strings for everything, including numbers, as long as I've been paying attention.We can merge this now or wait to see whether there's anything we can do to fix the rest of the tests, whatever you think makes sense.