fix(sdk): drop the invoice count from the delete blocker message - #1926
fix(sdk): drop the invoice count from the delete blocker message#1926whoAbhishekSah wants to merge 1 commit into
Conversation
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe plural unpaid-invoice blocker message no longer includes the invoice count. The related comment and test now reflect the count-free wording. ChangesUnpaid invoice blocker wording
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to For workspaces with multiple unpaid invoices, the deletion blocker now omits the invoice count while preserving the singular message. The updated behavior is covered without remaining merge-blocking risk. 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
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 |
Coverage Report for CI Build 34445219344Coverage remained the same at 50.229%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsNo coverage regressions found. Coverage Stats
💛 - Coveralls |
|
Closing. The screenshot that prompted this was taken from a tab still running the previous bundle. The current wording is fine as it is. |
When more than one invoice is unpaid, the delete blocker message now reads:
Please pay your outstanding invoices before deleting this workspace.It used to include the number, as in "your 3 outstanding invoices". The single-invoice sentence is unchanged.
🤖 Generated with Claude Code