kiteconnectjs icon indicating copy to clipboard operation
kiteconnectjs copied to clipboard

Exposing the parseBinary as a static method for 3rd party integrations?

Open darwin403 opened this issue 2 years ago • 3 comments

Context:

The ticker.ts lib has a rather elaborate and tedious way of handling the connection which may not suite for all use cases. For starters, you cannot set/change the access_token once a connection has been estabished since the package blocks the main thread. Therefore, although it might be useful for some cases, for any serious solution we should be able to handle the connection in a more customizable fashion.

Question

Most of the library is straight forward and we've used all the necessary variables that we require from the class but apart from the parseBinary method that follows a elaborate packet convention. It would immesenly help us out if we you can expose to this function as a static method.

Happy to make a PR, once you approve @vividvilla

Thanks! Sai

darwin403 avatar May 09 '23 10:05 darwin403

hey @darwin403, I've found the solution for this, and I plan to address it correctly by submitting a PR soon.

AbhishekCS3459 avatar Jun 11 '23 06:06 AbhishekCS3459

hey @darwin403 I am unable to generate the profile.json file when I am trying to test the code can you suggest how should I proceed?

AbhishekCS3459 avatar Jun 11 '23 10:06 AbhishekCS3459

go ahead with vitest end to end testing

Jatkingmodern avatar May 20 '24 12:05 Jatkingmodern