search

matbarofex / pyRofex

Python library to connect with Matba Rofex's Rest and Websocket APIs. Market Data and Order Routing are supported.
MIT License
83 stars 55 forks source link

Agrego nuevo entry ACP para poder suscribirse via WS #25

Closed roldan closed 2 years ago

fzanuso commented 2 years ago

Hola @roldan, como estas?

Estaba viendo el PR pero no estoy al tanto de este nuevo entry de MarketData, en la Doc de Primary tampoco veo que lo hayan agregado. Es algo que agregaron hace poco?

Saludos! Franco

roldan commented 2 years ago

Hola @fzanuso,

cómo va? Sí, hace unos días me avisaron de Primary que habían agregado este nuevo entry para bajar los precios de cierre al final de la rueda. Lo estuve probando y funciona perfecto, el tema es que no podía mandar el "ACP" por la validación que hay acá:

https://github.com/matbarofex/pyRofex/blob/536dd896d7f45dd066fe8ca31986ed1da011a942/src/pyRofex/service.py#L802

Saludos! Matías

fzanuso commented 2 years ago

Hable con la gente de Primary y me comentan que este entry es para MatrizOMS. Lo que voy a hacer es mergear este cambio y sacar la validacion de entries ya que no tiene mucho sentido hacerla en el conector. De esta manera si en el futuro agregan un entry nuevo no deberian haber problemas para utilizarlo.

roldan commented 2 years ago

buenísimo @fzanuso89! sí, te iba a decir que tal vez sacar la validación era lo mejor. Saludos y gracias por el laburo en este repositorio, muy útil!