feat(billing): show invoice descriptions and cap in-app list at 5#5747
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview The invoices API lowers The subscription API contract and billing settings UI pass through and render the new description in each row. Reviewed by Cursor Bugbot for commit 6e958c7. Configure here. |
Greptile SummaryThis PR adds invoice descriptions to the in-app billing history and reduces the visible list size. The main changes are:
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (3): Last reviewed commit: "fix(billing): decouple Stripe scan page ..." | Re-trigger Greptile |
|
@cursor review |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 6e958c7. Configure here.
Summary
description, falling back to the primary line item's description.Changes
app/api/billing/invoices/route.ts—expand: ['data.lines']on the Stripe list call, map adescriptionfield,MAX_INVOICES10 → 5.lib/api/contracts/subscription.ts— adddescription: string | nullto the invoice contract.billing.tsx— render the description in each row; add it to the row view-model.route.test.ts— updated the twoMAX_INVOICES-bound assertions and added coverage for description extraction/precedence.Type of Change
Testing
viteston the invoices route (5/5 passing, incl. new description test)check:api-validationpassesChecklist