Skip to content

feat: implement Scale order type#2083

Draft
dankim214 wants to merge 8 commits intomainfrom
dkim/scale-orders
Draft

feat: implement Scale order type#2083
dankim214 wants to merge 8 commits intomainfrom
dkim/scale-orders

Conversation

@dankim214
Copy link
Collaborator

Changes

Issue

Related Issues (Optional)

Screenshots/Recordings (Optional)

@vercel
Copy link

vercel bot commented Feb 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
v4-staging Ready Ready Preview, Comment Mar 2, 2026 7:42pm
v4-testnet Ready Ready Preview, Comment Mar 2, 2026 7:42pm

Request Review

@wiz-55df730c58
Copy link

wiz-55df730c58 bot commented Feb 24, 2026

Wiz Scan Summary

⚠️ Many findings detected
Many findings were detected, but only a subset of the findings are displayed inline due to API constraints. To view all findings inline, please click here.
Scanner Findings
Vulnerability Finding Vulnerabilities 25 High 18 Medium 6 Low
Data Finding Sensitive Data -
Secret Finding Secrets -
IaC Misconfiguration IaC Misconfigurations -
SAST Finding SAST Findings -
Software Management Finding Software Management Findings -
Total 25 High 18 Medium 6 Low

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

),
};
});
case TradeFormType.SCALE:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Can also merge this logic with case TradeFormType.LIMIT with some conditionals but claude recommends keeping this a separate code path which i can also see the merit for

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant