Refine strategy switch console options#11
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 849864d493
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| const strategyOptions = [ | ||
| "tqqq_growth_income", | ||
| "soxl_soxx_trend_income", |
There was a problem hiding this comment.
Bind SOXL strategy to its matching account
When an IBKR user leaves the default account selected (u15998061) and chooses this newly exposed SOXL/SOXX profile, buildInputs() still combines strategy_profile: form.strategy with the currently selected account's deployment_selector/account_scope/service_name, so the workflow dispatches soxl_soxx_trend_income to the live-u1599-tqqq service instead of the existing live-u1660-soxl account option. Please either make this profile select/require the SOXL account or constrain strategies per account before enabling dispatch.
Useful? React with 👍 / 👎.
Summary
Tests