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

Jli/cli pll claim evict #31

Merged
merged 6 commits into from
Apr 14, 2023
Merged

Jli/cli pll claim evict #31

merged 6 commits into from
Apr 14, 2023

Conversation

throwbackjams
Copy link
Contributor

Main components:

  • get seat manager info for a market
  • claim seat
  • evict seat with optional specific trader
  • update docs, including noting difference between claim seat and request seat

Cargo.toml Outdated Show resolved Hide resolved
Cargo.toml Outdated Show resolved Hide resolved
Copy link
Contributor

@jarry-xiao jarry-xiao 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, only a few minor comments

src/lib/processor/process_evict_seat.rs Show resolved Hide resolved
src/lib/processor/process_claim_seat.rs Show resolved Hide resolved
@throwbackjams throwbackjams merged commit 2c94ce1 into master Apr 14, 2023
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