Fix small futures account holdings json conversion bug (#30)

QuantConnect · Nov 28, 2023 · abc706b · abc706b
1 parent 17fd111
commit abc706b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/QuantConnect.BinanceBrokerage/BinanceFuturesRestApiClient.cs b/QuantConnect.BinanceBrokerage/BinanceFuturesRestApiClient.cs
@@ -86,7 +86,7 @@ public override List<Holding> GetAccountHoldings()
             }
 
             return JsonConvert
-                .DeserializeObject<FuturesAccountInformation>(response.Content, CreateAccountConverter())
+                .DeserializeObject<FuturesAccountInformation>(response.Content)
                 .Positions
                 .Where(p => p.PositionAmt != 0)
                 .Select(x => new Holding

diff --git a/QuantConnect.BinanceBrokerage/Messages/FuturesAccountInformation.cs b/QuantConnect.BinanceBrokerage/Messages/FuturesAccountInformation.cs
@@ -17,13 +17,13 @@
 
 namespace QuantConnect.BinanceBrokerage.Messages
 {
-    internal class FuturesAccountInformation
+    public class FuturesAccountInformation
     {
         public List<FutureBalance> Assets { get; set; }
         public List<FuturePosition> Positions { get; set; }
     }
 
-    internal class FuturePosition
+    public class FuturePosition
     {
         public  string Symbol { get; set; }
         public decimal PositionAmt { get; set; }