feat: more trading infos by parsing more from trading summary endpoint#57
Conversation
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #57 +/- ##
=======================================
Coverage 33.55% 33.55%
=======================================
Files 9 9
Lines 304 304
=======================================
Hits 102 102
Misses 202 202 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR extends trading summary parsing to include account-level details, exposing additional fields such as cash, activationDate, and pricingName via an optional account field on TradingSummaryItem.
- Add optional account: Option to TradingSummaryItem
- Update README macOS run instructions for quarantine removal and MFA note
- Bump package version to 0.4.0
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/bourso_api/src/client/trade/mod.rs | Adds account details to TradingSummaryItem alongside positions parsing. |
| README.md | Revises macOS app approval steps and clarifies MFA requirements. |
| Cargo.toml | Updates package version to 0.4.0 to reflect new features. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Add
accountoptional field inTradingSummaryItemwhich contains useful infos likecash,activationDate,pricingName, etc...