Skip to content

Updated the character counter direction#3433

Open
marcusljf wants to merge 1 commit intomainfrom
bounty-completion-character-count
Open

Updated the character counter direction#3433
marcusljf wants to merge 1 commit intomainfrom
bounty-completion-character-count

Conversation

@marcusljf
Copy link
Collaborator

@marcusljf marcusljf commented Feb 6, 2026

For some reason it was counting down, but it should be counting up to the limit.

Current Updated
CleanShot 2026-02-06 at 15 32 29@2x CleanShot 2026-02-06 at 15 30 40@2x

Summary by CodeRabbit

  • Bug Fixes
    • Updated the description length indicator in the bounty claim modal to display the current character count instead of remaining characters.

For some reason it was counting down, but it should be counting up to the limit.
@marcusljf marcusljf marked this pull request as ready for review February 6, 2026 23:33
@vercel
Copy link
Contributor

vercel bot commented Feb 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dub Error Error Feb 6, 2026 11:35pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 6, 2026

📝 Walkthrough

Walkthrough

The description length indicator in the bounty claim modal now displays the current description character count instead of the remaining available characters, altering the user-facing feedback mechanism.

Changes

Cohort / File(s) Summary
Description Length Display
apps/web/ui/partners/bounties/claim-bounty-modal.tsx
Modified the character count display to show current description length (description.length) instead of remaining characters (max - description.length).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • PR #3049: Adds max-length constant and character limit enforcement to the same modal file, complementing this change by providing the underlying constraint mechanism for the length display.

Suggested reviewers

  • steven-tey

Poem

🐰 A counter once counted what's left,
Now it counts what it has in its chest,
From "five characters free" to "twenty used",
Progress revealed, no confusion abused! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and accurately describes the main change: updating the character counter to count up to the limit rather than counting down.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bounty-completion-character-count

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.

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.

1 participant