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.

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

Add is active endpoint #2256

Merged

Conversation

mstur
Copy link
Contributor

@mstur mstur commented May 14, 2021

Description
Adds a check of a user is either subscribed or trialed.

Tests Written / Performed
Added test with sample user and cleaned up as best as possible (not able to delete a price and therefore not a product either – see here stripe/stripe-python#658)

"""
client = StripeClient(current_app.config["STRIPE_SECRET"])

if client.is_paying(customer_id) or client.is_trialed(customer_id):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

return client.is_paying(customer_id) or client.is_trailed(customer_id) :)

@janniezhong janniezhong force-pushed the mia/main-webserver/stripe-endpoints branch from c1b06f1 to eb4fc14 Compare May 17, 2021 20:02
@mstur mstur requested a review from a team May 17, 2021 20:02
@janniezhong janniezhong force-pushed the mia/main-webserver/stripe-endpoints branch from eb4fc14 to db5867f Compare May 17, 2021 20:30
lithafnium and others added 5 commits May 17, 2021 13:49
moved into helpers (#1995)

Revert "moved into helpers (#1995)" (#1996)

This reverts commit 4cbf261.

oops

remove unneeded, put back needed
testing a little broken sad
remove excess

some weird auth thing left
cleanup

spacing

one moreee

clean stripe

clean up
@janniezhong janniezhong force-pushed the mia/main-webserver/stripe-endpoints branch from db5867f to f0b17e1 Compare May 17, 2021 20:51
@janniezhong janniezhong merged commit eead84d into project/webserver-payments May 17, 2021
@janniezhong janniezhong deleted the mia/main-webserver/stripe-endpoints branch May 17, 2021 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants