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

Send take #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Send take #1

wants to merge 2 commits into from

Commits on Oct 23, 2022

  1. process_send_take function implemented (#251) 

    * send-take implemented

* send take

* remove comments, return err

* Replace 2 more checks with is_send_take flag

* attempted cargo fix

* Variable cleanup and code modification to handle fee updates in matching.rs

* Remove unused variable and add assert

* revert Cargo.lock

* Anchor version

* revert anchor version to 0.18.2

* update CI script

* cargo.lock

* Added test cases to more throroughly stress test SendTake

* Remove unnecessary variable in test

* Revert PR changes related to build

* Skip permissioned markets test

* Fix indentation

* cargo fmt the state.rs file

* Add overflow checks

* variable rename

* open orders accounting seems correct

* Added print statements to understand the SendTake test case

* get rid of overflow checks and calculate fees in matching

Co-authored-by: jarry-xiao <jarry.xiao@gmail.com>
    @a-creation @jarry-xiao
    a-creation and jarry-xiao authored Oct 23, 2022
    Configuration menu
    Copy the full SHA
    4713d2f View commit details
    Browse the repository at this point in the history

  2. feat: add send_take to instruction.rs to facilitate CPI (#257)

    @kootsZhin
    kootsZhin authored Oct 23, 2022
    Configuration menu
    Copy the full SHA
    d678cea View commit details
    Browse the repository at this point in the history