Skip to content

Conversation

@tompntn
Copy link
Contributor

@tompntn tompntn commented Jul 5, 2022

This PR demonstrates how to use C++ to publish price updates to the Pyth Network through the pythd websocket API described at https://docs.pyth.network/publish-data/pyth-client-websocket-api

It's a very rough, minimal example which demonstrates how to use the API.

@tompntn tompntn requested review from Reisen and jayantk July 5, 2022 16:08
Copy link
Contributor

@jayantk jayantk left a comment

Choose a reason for hiding this comment

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

the high-level approach makes sense to me. I left you some more tactical comments.

@tompntn tompntn force-pushed the test-publish-cpp-websockets branch from e7bde6c to 1516324 Compare July 6, 2022 14:17
@tompntn tompntn marked this pull request as ready for review July 11, 2022 09:14
Copy link
Collaborator

@ali-behjati ali-behjati left a comment

Choose a reason for hiding this comment

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

I don't have c++ not qt expertise here but the code is very nice and clear and covers all use cases that comes to my mind.

:shipit:

@tompntn tompntn merged commit 5b79908 into main Jul 11, 2022
@tompntn tompntn deleted the test-publish-cpp-websockets branch July 11, 2022 09:48
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.

4 participants