feat(subscription): 优化凭据导入与账号卡片 - #456
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: QUIET Plan: Pro Plus Run ID: 📒 Files selected for processing (11)
💤 Files with no reviewable changes (1)
Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthrough本次变更调整订阅账号额度卡片、运行概况和加载骨架。连接抽屉按钮统一使用紧凑尺寸。订阅凭据导入流程新增 OAuth JSON 文件入口，并调整 JSON 粘贴面板、渠道通知和风险提示条件。英文、日文和中文本地化文本同步更新。 Merge Risk: ⚪ Minimal · up to This PR changes subscription credential-management UI and translations without introducing an identified correctness, data, security, or availability risk; it is merge-ready after normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches✨ Simplify code
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
🟢 Approval recommended
Changes are localized to frontend UI/i18n, removed keys are no longer referenced, and the updated components remain internally consistent with the new translation keys.
Pull request overview
This PR refines the subscription credential import flow and subscription account card presentation in the web UI, focusing on reducing redundant guidance text and improving information density while keeping backend/API behavior unchanged.
Changes:
- Simplifies subscription credential import copy and reorganizes entry actions (authorize + credential file import, with JSON paste as a secondary disclosure).
- Updates subscription account card quota display to a flatter, single-line meter and reshapes the expanded area into a “runtime overview” layout.
- Adjusts i18n strings (zh-CN / en-US / ja-JP) to match the new UI structure, including new/renamed labels.
File summaries
| File | Description |
|---|---|
| web/src/i18n/locales/zh-CN/import.ts | Updates subscription import strings (risk notice, paste JSON, file import label) and removes deprecated copy. |
| web/src/i18n/locales/zh-CN/group.ts | Renames/adjusts subscription card labels (daily summaries, quota reset prefix/suffix, overview heading). |
| web/src/i18n/locales/ja-JP/import.ts | Mirrors subscription import string updates for Japanese. |
| web/src/i18n/locales/ja-JP/group.ts | Mirrors subscription card label updates for Japanese. |
| web/src/i18n/locales/en-US/import.ts | Mirrors subscription import string updates for English. |
| web/src/i18n/locales/en-US/group.ts | Mirrors subscription card label updates for English. |
| web/src/features/import/SubscriptionCredentialStager.vue | Reworks subscription credential entry UI and feedback placement; updates related copy keys and styling. |
| web/src/features/import/NewGroupImport.vue | Makes credential step description conditional (omits subscription description). |
| web/src/features/import/ChannelPresetPicker.vue | Removes subscription-risk inline notice from preset picker (now handled in the connect area). |
| web/src/features/groups/credentials/SubscriptionAccountCard.vue | Replaces quota progress component with inline meter + revised quota/reset text; reshapes expanded detail into runtime overview. |
| web/src/features/groups/credentials/GroupCredentialsTab.vue | Small modal copy/size tweaks (inline description, compact footer buttons). |
Review details
- Files reviewed: 11/11 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 049b33d041
ℹ️ 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".
| .subscription-account__quota-meta { | ||
| position: relative; | ||
| z-index: 1; | ||
| display: inline-flex; | ||
| min-width: max-content; | ||
| align-items: center; |
There was a problem hiding this comment.
When a quota has a long localized value or reset label, this min-width: max-content makes the entire right-hand block unshrinkable, while .subscription-account__quota uses overflow: hidden. On narrow account cards, the remaining quota and reset time are therefore clipped with no tooltip or wrapping fallback. Allow this block to wrap/shrink, or restore the previous responsive second row for the reset value.
Useful? React with 👍 / 👎.
|
远程 head 分支已按本地 Codex 配置重命名为 |
关联 Issue / Related Issue
无（订阅凭据管理前端交互优化）
变更内容 / Change Content
自查清单 / Checklist
make check，或在说明中写明无法运行的原因和未验证范围。 / I ranmake check, or documented why it could not run and what remains unverified.无需更新公开文档或执行数据迁移；本次仅调整管理前端交互与三语文案。
Summary by CodeRabbit
新功能
优化