Skip to content

Fix USD symbol currency detection in total P&L#96

Open
dbigham wants to merge 2 commits into
mainfrom
codex/investigate-eli-account-total-pl-discrepancies
Open

Fix USD symbol currency detection in total P&L#96
dbigham wants to merge 2 commits into
mainfrom
codex/investigate-eli-account-total-pl-discrepancies

Conversation

@dbigham

@dbigham dbigham commented Oct 11, 2025

Copy link
Copy Markdown
Owner

Summary

  • add symbol currency inference so USD tickers no longer default to CAD when activities report CAD
  • prefer Questrade symbol details when available to correct total P&L currency handling
  • add a unit test that mocks Questrade and Yahoo responses to cover the USD security scenario

Testing

  • npm test

https://chatgpt.com/codex/tasks/task_e_68eabeb4e9a0832d953aa3e6bb5659d0

@dbigham
dbigham force-pushed the main branch 3 times, most recently from c7f2145 to c8636e3 Compare November 15, 2025 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant