Skip to content

Conversation

@reizhafajrian
Copy link

i just add subscribe with symbol and remove unused code

@reizhafajrian
Copy link
Author

i think we can create dev branch for testing it,before merge to main branch

@endenwer
Copy link
Owner

endenwer commented Oct 3, 2022

Thanks for pr, but I can't accept it as is. There is no tests, some strange files like src/TradingViewWebsocket.ts. And It is better to make different pr's for refactoring and new features.

@reizhafajrian
Copy link
Author

Thanks for pr, but I can't accept it as is. There is no tests, some strange files like src/TradingViewWebsocket.ts. And It is better to make different pr's for refactoring and new features.

i think we can create new branch like dev branch

Copy link

@CODLOP CODLOP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how we can get ticks if callback is not disponible !!

volume: event.params.p[1].sds_1.s[0].v[5],
symbol
}
// callback(candles)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i don't know why you comment the callback !!

])
connection.subscribe((event: any) => {
if (event.name = "event") {
if (event.params.p[1].sds_1?.s !== undefined) {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all event.params.p should be event.params

])


export async function connectAndSubscribe({ connection, symbols, timeframe = 1 }: GetCandlesParams) {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where callback param !!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants