feat: Expand Credit Card feature into full Credits system with payment schedules #199
No reviewers
Labels
No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
nidde/parenting-tool!199
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "copilot/expand-credit-card-feature"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Description
Expands the Credit Cards feature into a comprehensive Credits system supporting multiple credit types (Credit Cards, Klarna, Other creditors) with payment schedule generation, tracking, and ledger/balance integration.
Backend
CreditType(credit_card, klarna, other) andPaymentType(full, split)credit_cards→creditstable with new columns; newcredit_paymentstableCredit(replaces CreditCard) andCreditPaymentwith computed attributes (amount_per_payment, total_paid, remaining_balance, is_paid_off)Frontend
/credits): Type-aware form dialog, payment schedule viewer, credit card calculator (retained for CC type), credit summary (total before/after)Testing
BalanceTestto pass the newCreditServicedependency as 4th constructor parameter toBalanceServiceType of Change
Testing
Code Quality
CI/CD Checks
Additional Notes
@copilot
Seems your changes didn't include updating the Balance tests, as several of them are now failing, expecting 4 parameters but only getting 3
Fixed in
73e0158. TheBalanceTestwas manually constructingBalanceServicewith only 3 parameters, but the constructor now requires 4 (addedCreditService). Added the missingCreditServiceimport and parameter. All 15 balance tests pass now.Comment
@coderabbitai helpto get the list of available commands and usage tips.LG2M 👍