Skip to content

fix: update warning threshold label from '5$' to '2$' - #2690

Merged
seefs001 merged 1 commit into
QuantumNous:mainfrom
feitianbubu:pr/0d926e8180210062b85a4ee06a0b324ba9ec91f6
Jan 20, 2026
Merged

fix: update warning threshold label from '5$' to '2$'#2690
seefs001 merged 1 commit into
QuantumNous:mainfrom
feitianbubu:pr/0d926e8180210062b85a4ee06a0b324ba9ec91f6

Conversation

@feitianbubu

@feitianbubu feitianbubu commented Jan 19, 2026

Copy link
Copy Markdown
Member

修复个人设置-额度预警阈值等价金额5$显示错误

Summary by CodeRabbit

  • Bug Fixes
    • Corrected a label in notification settings to display the accurate warning threshold value.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai

coderabbitai Bot commented Jan 19, 2026

Copy link
Copy Markdown
Contributor

Walkthrough

A notification settings component receives a minor UI label update. The display text for a warning threshold option changed from '5$' to '2$', with no functional logic or validation modifications.

Changes

Cohort / File(s) Summary
Notification Settings UI Label Update
web/src/components/settings/personal/cards/NotificationSettings.jsx
Updated warningThreshold field option label from '5$' to '2$' for value 1000000. Purely cosmetic UI change with no logic impact.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • seefs001

Poem

🐰 A label was wrong, now it's right,
From five to two dollars in sight,
UI polish so fine,
Makes the threshold align,
Details matter—we caught it just right! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately describes the main change: updating a warning threshold label from '5$' to '2$' in the notification settings.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@seefs001
seefs001 merged commit 57ed2b3 into QuantumNous:main Jan 20, 2026
1 check passed
ennnnny pushed a commit to ennnnny/new-api that referenced this pull request Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

个人设置-额度预警阈值等价金额是2刀的显示不对

2 participants