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

Deposit limits #806

Merged
merged 1 commit into from
Dec 5, 2023
Merged

Deposit limits #806

merged 1 commit into from
Dec 5, 2023

Conversation

ckamm
Copy link
Contributor

@ckamm ckamm commented Dec 5, 2023

  • limit deposits (via deposit, flash loan, tcs)
  • limit potential deposits via openbook settle by restricting placable orders via potential_serum_tokens
  • introduce Serum3PlaceOrderV2 for this purpose
  • account for new limits in liquidator, max_swap

@ckamm ckamm requested a review from microwavedcola1 December 5, 2023 12:15
Base automatically changed from ckamm/r21-ob-tracking to dev December 5, 2023 14:39
- limit deposits (via deposit, flash loan, tcs)
- limit potential deposits via openbook settle
  by restricting placable orders via potential_serum_tokens
- introduce Serum3PlaceOrderV2 for this purpose
- account for new limits in liquidator, max_swap
@ckamm ckamm force-pushed the ckamm/r21-deposit-limit branch from a98efc7 to 27f0fc6 Compare December 5, 2023 14:42
@github-actions github-actions bot added program On-chain program changes client TS client changes labels Dec 5, 2023
@ckamm ckamm merged commit 42e31ae into dev Dec 5, 2023
23 checks passed
@ckamm ckamm deleted the ckamm/r21-deposit-limit branch December 5, 2023 14:43
ckamm added a commit that referenced this pull request Dec 5, 2023
- limit deposits (via deposit, flash loan, tcs)
- limit potential deposits via openbook settle
  by restricting placable orders via potential_serum_tokens
- introduce Serum3PlaceOrderV2 for this purpose
- account for new limits in liquidator, max_swap

(cherry picked from commit 42e31ae)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client TS client changes program On-chain program changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants