feature: PiggyCards fixed denomination cards and version bump to 8.5.1#740
feature: PiggyCards fixed denomination cards and version bump to 8.5.1#740bfoss765 merged 2 commits intofeature/piggycardsfrom
Conversation
- Aggregate denominations from ALL gift cards returned by PiggyCards API (Netflix, Amazon, eBay, etc. return separate cards per denomination) - Track discount per denomination instead of using highest discount - Update savingsFraction when user selects a denomination - Fix showCost to display cost message for fixed denomination cards - Update PercentageFormatter to show decimals when value has fraction - Fix confirmation dialog to use PercentageFormatter instead of intValue Fixes issues where: - Only one denomination showed (e.g., Netflix only $25 instead of $20/$25/$50) - Wrong discount displayed (highest instead of per-card discount) - Cost message not showing for fixed denomination cards - Percentages rounded incorrectly (13.5% showing as 14% or 13%) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
|
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 You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Tip 📝 Customizable high-level summaries are now available in beta!You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.
Example instruction:
Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later. 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 |
Summary
Changes
DashSpendPayViewModel.swift
denominationDiscountsdictionarysavingsFractionwhen user selects a denominationshowCostto display cost message for fixed denomination cardsDashSpendConfirmationDialog.swift
PercentageFormatterinstead of.intValuefor discount displayPercentageFormatter.swift
project.pbxproj
Test plan
🤖 Generated with Claude Code