Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds fills websocket feed #100

Merged

Conversation

hoakbuilds
Copy link
Member

Status Type ⚠️ Core Change Issue
Ready Feature No #95

Problem

What problem are you trying to solve?

See #95

Solution

How did you solve the problem?

Adds the fills websocket feed from mango

Squashed commits:

[5189ee7] WIP: Adds fills websocket feed

(cherry picked from commit 0c7bb97916347684a16d25dd49452ef4d128dcc6)
@hoakbuilds hoakbuilds changed the title Adds the decoded events to the fills feed Adds fills websocket feed Apr 25, 2022
@hoakbuilds hoakbuilds merged commit d6af986 into bmresearch:net5 Apr 25, 2022
@coveralls
Copy link

coveralls commented Apr 25, 2022

Pull Request Test Coverage Report for Build 2221618547

  • 4 of 77 (5.19%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-2.4%) to 78.051%

Changes Missing Coverage Covered Lines Changed/Added Lines %
Solnet.Mango.Historical/Models/FillsSnapshot.cs 0 5 0.0%
Solnet.Mango.Historical/Models/FillsEvent.cs 0 7 0.0%
Solnet.Mango.Historical/MangoHistoricalDataService.cs 4 65 6.15%
Totals Coverage Status
Change from base Build 2110522868: -2.4%
Covered Lines: 2458
Relevant Lines: 3082

💛 - Coveralls

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.

2 participants