Merge pull request #273 from utekaravinash/historic-trades-limited-to…

…-20k-with-duplicates Bug Fix: historic trades limited to 20k with duplicates
alpacahq · Dec 12, 2019 · 18cfb44 · 18cfb44
2 parents 4e2fd27 + 37dac49
commit 18cfb44
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/contrib/polygon/api/api.go b/contrib/polygon/api/api.go
@@ -176,7 +176,6 @@ func GetHistoricTrades(symbol, date string) (totalTrades *HistoricTrades, err er
 		resp   *http.Response
 		u      *url.URL
 		q      url.Values
-		trades = &HistoricTrades{}
 	)
 
 	for {
@@ -206,6 +205,7 @@ func GetHistoricTrades(symbol, date string) (totalTrades *HistoricTrades, err er
 			return nil, fmt.Errorf("status code %v", resp.StatusCode)
 		}
 
+		trades := &HistoricTrades{}
 		if err = unmarshal(resp, trades); err != nil {
 			return nil, err
 		}