JKorf / FTX.Net

MIT License
31 stars 24 forks source link

Deserialize error in TradeApi.Account.GetDepositHistoryAsync #27

Closed CitrusVento closed 2 years ago

CitrusVento commented 2 years ago

-1 [558] Deserialize JsonSerializationException: Error converting value {null} to type 'System.DateTime'. Path 'result[0].confirmedTime', line 1, position 383. data: {\r\n \"success\": true,\r\n \"result\": [\r\n {\r\n \"id\": 19068986,\r\n .... \"status\": \"unconfirmed\",\r\n \"time\": \"2022-04-15T11:13:47.444108+00:00\",\r\n \"sentTime\": \"2022-04-15T11:13:04+00:00\",\r\n \"confirmedTime\": null,\r\n \"confirmations\": 11,\r\n \"method\": \"arbitrum\"\r\n }\r\n ]\r\n}

For unconfirmed depositis confirmedTime is null. Type of ConfirmTime property of FTXDeposit should be 'DateTime?'

JKorf commented 2 years ago

Fixed in latest version