fix(trade): clear margin requirement copy in the order receipt - #2422
Conversation
The row labeled "Margin req." actually displayed account capital -> capital-minus-margin (a balance readout), so the actual requirement never appeared as a number and the label misdescribed the row. Split it: "Margin" now shows what this order reserves, and "Available to trade" shows before -> after on the same available basis as the account strip (capital minus already-locked margin) so the two readouts always agree. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Item 2 of the terminal UX fixes. The receipt row labeled Margin req. actually showed
capital → capital − margin— a balance readout — so the real requirement never appeared and the label misdescribed the number. Now:tsc clean; no logic beyond the two display rows changed.
🤖 Generated with Claude Code