8000 feat: bitfinex ws by aljo242 · Pull Request #45 · skip-mev/connect · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Mar 24, 2025. It is now read-only.

feat: bitfinex ws #45

Merged
merged 23 commits into from
Jan 26, 2024
Merged

feat: bitfinex ws #45

merged 23 commits into from
Jan 26, 2024

Conversation

aljo242
Copy link
Contributor
@aljo242 aljo242 commented Jan 26, 2024

Closes BLO-734

Use channel map to track state. BitFinex stream messages are not json encoded, so a lot of interface conversion needs to be done.

See bitfinex implementation for similar logic.

Copy link
codecov bot commented Jan 26, 2024

Codecov Report

Attention: 71 lines in your changes are missing coverage. Please review.

Comparison is base (31fc9d8) 72.50% compared to head (390835d) 72.09%.

Files Patch % Lines
providers/websockets/bitfinex/ws_data_handler.go 65.51% 22 Missing and 8 partials ⚠️
providers/websockets/bitfinex/messages.go 34.28% 23 Missing ⚠️
providers/websockets/bitfinex/parse.go 66.66% 15 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
- Coverage   72.50%   72.09%   -0.42%     
==========================================
  Files         108      111       +3     
  Lines        5802     5978     +176     
==========================================
+ Hits         4207     4310     +103     
- Misses       1307     1369      +62     
- Partials      288      299      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Alex Johnson and others added 4 commits January 26, 2024 11:30
Co-authored-by: David Terpay <35130517+davidterpay@users.noreply.github.com>
aljo242 added 2 commits January 26, 2024 11:42
Co-authored-by: David Terpay <35130517+davidterpay@users.noreply.github.com>
bet
< 86DF input type="hidden" name="dropdown_direction" value="s" autocomplete="off" data-targets="batch-deferred-content.inputs" />
@aljo242 aljo242 merged commit 7bc1131 into main Jan 26, 2024
@aljo242 aljo242 deleted the feat/bitfinex-ws branch January 26, 2024 17:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0