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.

Sign up for GitHub

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

Jump to bottom

feat: market liquidities account #160

Merged
merged 7 commits into from
Dec 4, 2023
Merged

feat: market liquidities account #160

merged 7 commits into from
Dec 4, 2023

Conversation

pawel-betdex
Copy link
Contributor

@pawel-betdex pawel-betdex commented Nov 29, 2023 •
edited
Loading

Second account needed to deliver improved matching. This account holds summary of the unmatched liquidity available on the market.

obi-ewan
obi-ewan previously approved these changes Dec 1, 2023
View reviewed changes
eoin-betdex
eoin-betdex requested changes Dec 1, 2023
View reviewed changes
Copy link
Contributor

@eoin-betdex eoin-betdex left a comment

Choose a reason for hiding this comment

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

Looks good I think. Just maybe a copy and paste error left in get_liquidity_against

programs/monaco_protocol/src/state/market_liquidities.rs Outdated Show resolved Hide resolved
programs/monaco_protocol/src/state/market_liquidities.rs Outdated Show resolved Hide resolved
@pawel-betdex
Copy link
Contributor Author 

Test Suites: 75 passed, 75 total
Tests:       341 passed, 341 total
Snapshots:   0 total
Time:        507.484 s

@pawel-betdex pawel-betdex merged commit a753f36 into MonacoProtocol:develop Dec 4, 2023
@pawel-betdex pawel-betdex deleted the feat/market-liquidities-account branch December 4, 2023 10:47
@eoin-betdex eoin-betdex mentioned this pull request Feb 26, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eoin-betdex eoin-betdex eoin-betdex approved these changes

@obi-ewan obi-ewan obi-ewan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pawel-betdex @obi-ewan @eoin-betdex