Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Client update needed to match WebSocket spec changes #326

Closed
client-library-maintainer bot opened this issue Nov 14, 2022 · 1 comment
Closed

Client update needed to match WebSocket spec changes #326

client-library-maintainer bot opened this issue Nov 14, 2022 · 1 comment
Labels
spec The client may need an update due to API spec changes

Comments

@client-library-maintainer
Copy link

A diff between this client library's spec and our hosted spec was found. The client may need an update to match any changes in our API. See the diff below:

--- https://raw.githubusercontent.com/polygon-io/client-python/master/.polygon/websocket.json
+++ https://api.polygon.io/specs/websocket.json
@@ -185,6 +185,14 @@
                     "q": {
                       "type": "integer",
                       "description": "The sequence number represents the sequence in which message events happened.\nThese are increasing and unique per ticker symbol, but will not always be\nsequential (e.g., 1, 2, 6, 9, 10, 11).\n"
+                    },
+                    "trfi": {
+                      "type": "integer",
+                      "description": "The ID for the Trade Reporting Facility where the trade took place."
+                    },
+                    "trft": {
+                      "type": "integer",
+                      "description": "The TRF (Trade Reporting Facility) Timestamp in Unix MS. \nThis is the timestamp of when the trade reporting facility received this trade.\n"
                     }
                   }
                 },
@@ -2247,6 +2255,14 @@
           "q": {
             "type": "integer",
             "description": "The sequence number represents the sequence in which message events happened.\nThese are increasing and unique per ticker symbol, but will not always be\nsequential (e.g., 1, 2, 6, 9, 10, 11).\n"
+          },
+          "trfi": {
+            "type": "integer",
+            "description": "The ID for the Trade Reporting Facility where the trade took place."
+          },
+          "trft": {
+            "type": "integer",
+            "description": "The TRF (Trade Reporting Facility) Timestamp in Unix MS. \nThis is the timestamp of when the trade reporting facility received this trade.\n"
           }
         }
       },
@client-library-maintainer client-library-maintainer bot added the spec The client may need an update due to API spec changes label Nov 14, 2022
@danielatpolygonio
Copy link
Contributor

Also fixed by #334

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec The client may need an update due to API spec changes
Projects
None yet
Development

No branches or pull requests

1 participant