fix(ui): simplify strategy display — markets only, clean i18n - #174
Merged
Conversation
Problem: strategyDisplayMetaText showed raw technical fields
(lifecycle_stage, runtime_enabled, can_switch_live, blocked_live_reason,
allowed_execution_modes, latest_evidence_status, plugin_gate_status) in
hardcoded English below the strategy dropdown. Useless for end users and
inconsistent with the Chinese UI.
Also strategyChoiceLabel used cryptic tags like [blocked live] (typo)
and [disabled] in the dropdown.
Changes:
- strategyDisplayMetaText: just show supported markets via t("strategyMeta")
- strategyChoiceLabel: show domain label (eg. 美股) instead of [blocked live]/[disabled]
- strategyMeta i18n: zh → "支持市场：{domains}", en → "Markets: {domains}"
- Remove dead strategyBlockedLiveReason() function (47 lines)
Co-Authored-By: Claude <noreply@anthropic.com>
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
🤖 Codex PR ReviewPlease ensure a human reviewer checks this PR before merging. |
Co-Authored-By: Claude <noreply@anthropic.com>
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
问题
策略下拉框下方的提示文字显示了一堆原始技术字段：
这些信息对最终用户无价值，且全是硬编码英文，与中文 UI 不一致。
下拉选项中 disabled/blocked 策略也显示
[blocked live]（有拼写错误）和[disabled]等技术标签。改动
strategyDisplayMetaText支持市场：美股 / 港股strategyChoiceLabel纳斯达克增长收益 [disabled]纳斯达克增长收益（美股）该账号仅显示 {domains} 策略支持市场：{domains}This account only shows {domains} strategiesMarkets: {domains}strategyBlockedLiveReason()净删除 47 行，新增 8 行。
🤖 Generated with Claude Code