Skip to content

Comments

✅301 acceptance standardize cut order logic#329

Open
ericsharma wants to merge 14 commits intomainfrom
✅301-acceptance-standardize-cut-order-logic

Hidden character warning

The head ref may contain hidden characters: "\u2705301-acceptance-standardize-cut-order-logic"
Open

✅301 acceptance standardize cut order logic#329
ericsharma wants to merge 14 commits intomainfrom
✅301-acceptance-standardize-cut-order-logic

Conversation

@ericsharma
Copy link
Collaborator

ℹ Overview

  • Cut order logic standardized to limit escrows to minimum balance of 0.5 algo meaning that a taker order will now either closeout an escrow order entirely or leave it open with at least a total of 0.5 algo.
  • When in execution:both if a taker closes out all escrows at desired price and still hasn't achieved the initial desired amount, a maker order will only be placed if the total algo amount is greater than 0.5 algo .
  • Stabilized the fee for order type: buy so desired ASA amount ALWAYS matches what the user inputted. When adding to an orderbook a fee of 0.001 algo is taken directly from the algo total so by adding an extra 0.001 algo to the total, the original asa amount is preserved. For selling ASA's the fees for adding to the orderbook are taken out of the 0.5 algo provided in the maker order.
  • After extensive testing, I have removed the txn-fee calculation from cutOrder. I have yet to run into any issues related to taking out this fee and in removing the fee, cutOrder amounts are much more consistent with what the user might expect.

📝 Related Issues

#302
#303
#304
#305

…nces-under-05-algo

302 add detection of escrow balances under 0.5 algo
…scrow-orders

✅Encapsulated restrict order logic and added algo escrow side
…scrow-asa-amount-less-than-user-inputted-asa-amount-for-maker-buy
…user-inputted-asa-amount-for-maker-buy

305 escrow asa amount less than user inputted asa amount for maker buy
@ericsharma ericsharma requested a review from atrefonas October 19, 2022 14:25
@ericsharma ericsharma linked an issue Oct 19, 2022 that may be closed by this pull request
4 tasks
@vercel
Copy link

vercel bot commented Oct 19, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
algodex-sdk ✅ Ready (Inspect) Visit Preview Oct 19, 2022 at 2:38PM (UTC)

Copy link
Contributor

@atrefonas atrefonas 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 to me.

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.

Acceptance: Standardize cut order logic

2 participants