Repository navigation
Conversation
The pending-transfer line read "expires 2026-09-24if it isn't accepted": JSX drops the space between an expression and the word after it when the two straddle a line break, so the expiry date ran into the following word. Build the sentence as one string in `transferPendingNote` and cover the spacing with a unit test, so reflowing the JSX can't reintroduce it. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
|
Landed directly on staging as ade65b3 instead. |
Follow-up to #248. The pending-transfer line rendered as:
JSX drops the space between an expression and the word after it when the two straddle a line break, so the expiry date ran into the following word — the space is present in the source, which is what makes this easy to reintroduce while reformatting.
Builds the sentence as one string in a
transferPendingNotehelper and pins the spacing with a unit test, following the existing pure-helper testing pattern in this component.Also deletes the branch-recreation artifact: #248 was merged while this fix was in flight, so it lands on a fresh branch off current
staging.npx tsc --noEmit,npm run lintand the group component tests are clean.🤖 Generated with Claude Code