Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add Support for Tiered Pricing Plans #629

Open
wants to merge 14 commits into
base: original
Choose a base branch
from
Open

Add Support for Tiered Pricing Plans #629

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
471de42
Implementation without Tests
jksimoniii Feb 1, 2019
76cc607
Add Test Cases for Models
jksimoniii Feb 1, 2019
766ed1e
Handle null Amounts in convert_amount_for_db
jksimoniii Feb 1, 2019
f125831
Add Tests for Syncing Plans with Tiers
jksimoniii Feb 1, 2019
271ddfb
Add Tests for ConvertAmountForDBTests
jksimoniii Feb 1, 2019
2c33421
Add Tests for TieredPricingManagerTests
jksimoniii Feb 1, 2019
eb0492f
Fix Failing TieredPricingManagerTests
jksimoniii Feb 1, 2019
b323f9e
Update Tier to Use Decimal Fields
jksimoniii Feb 1, 2019
cbfa6e3
Fix Flake Errors
jksimoniii Feb 1, 2019
0a75829
Fix Django Compatability Issues
jksimoniii Feb 1, 2019
db37888
Use Plan Total Calculator for Subscriptions
jksimoniii Feb 3, 2019
421bb78
Update Migrations
jksimoniii Feb 3, 2019
17a915b
Update Migrations
jksimoniii Feb 3, 2019
4104c7d
Increase Code Coverage
jksimoniii Feb 3, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Handle null Amounts in convert_amount_for_db
  • Loading branch information
@jksimoniii
jksimoniii committed Feb 1, 2019
commit 766ed1e8d07619eac9d4db2ab3d0d49a3d3c5762
2 changes: 2 additions & 0 deletions pinax/stripe/utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,8 @@ def convert_tstamp(response, field_name=None):
def convert_amount_for_db(amount, currency="usd"):
if currency is None: # @@@ not sure if this is right; find out what we should do when API returns null for currency
currency = "usd"
if amount is None: # @@@ not sure if this is right; find out what we should do when API returns null for amount
amount = 0
return (amount / decimal.Decimal("100")) if currency.lower() not in ZERO_DECIMAL_CURRENCIES else decimal.Decimal(amount)


Expand Down