fix(frontend): auto-select newly created strategy in sidebar #561
Conversation
|
Thank you for your continuous contributions! We really appreciate your consistency and effort recently. Feel free to join our Discord server if you need real-time communication or just want to chat. |
Just joined, looking forward to helping out more :) |
|
Thanks for the PR! Regarding the implementation, I'd prefer not to introduce an internal state like previousStrategyIds to track changes. Instead, we should rely on Routing as the single source of truth for the currently selected strategy. Suggested approach: Ensure the selected strategy is always derived from the URL (e.g., /strategies/:strategyId). When a new strategy is created, simply trigger a navigation to the new route (redirect to the new strategyId). This approach keeps the state management cleaner and avoids synchronization issues between local state and the UI. |
PotatoZhou
changed the title
* 'main' of github.com:PotatoZhou/valuecell: chore: temporarily switch to TradingView for all stock data rendering (#572) feat: enhance model validation and API key input UI (#571) fix(tracker): split useEffect hooks avoid resend, and dynamically retrieve tracker user_id (#569) feat: add TradingView plugin to display the index and stock information (#511)
PotatoZhou
changed the title
3 participants
📝 Pull Request Template
1. Related Issue
Closes #517
2. Type of Change (select one)
Type of Change: Bug Fix
3. Description
Add previousStrategyIds state to track strategy changes and Automatically select new strategies when detected
4. Testing
5. Checklist