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

Stripe billing and invoicing #180

Open
wants to merge 64 commits into
base: master
Choose a base branch
from

Conversation

bohdan-vorona
Copy link
Contributor

@bohdan-vorona bohdan-vorona commented Apr 12, 2023

It's a DRAFT PR since there are some open questions about customers here - #174 (comment).

Docs -> intro-docs/subscriptions.md.

Params:

    /**
     * Stripe
     */
    'stripe' => [
        'publishable_key' => 'pk_test_UdowAZ9Ju8kv7WWAHoG4UzbT',
        'secret_key' => 'sk_test_mjCzZMDmDwcpbOG4DusbUDdx',
        'pricing_table_id' => 'prctbl_1MmNdnAVtR6J0zbouXceXi2G',
        'customer_portal_url' => 'https://billing.stripe.com/p/login/test_cN24i5ffoeQQg5qaEE',
        'webhook_key' => 'we_1MslXGAVtR6J0zbomWCMuZ8R',
        'webhook_signing_secret' => 'whsec_3INA6CBeM79n00nqAwPMGmDpqaIiMRn7',
    ],

stripe-manage


stripe-redirect


stripe-thx


stripe-menu


stripe-subscribe


stripe-active


stripe-trial


stripe-usage

…ut.session.completed` dummy job implementation
…, updated SubscriptionService, updated StripeCheckoutSessionCompletedJob
…ge, subscription page, Subscription History upgrade, SubscriptionService upgrade, StripeCheckoutSessionCompletedJob update
…riptionDeletedJob, BaseStripeJob upgrade, updated previous Jobs, updated docs
…b, table update, StripeCustomerSubscriptionUpdatedJob
@bohdan-vorona bohdan-vorona added the enhancement New feature or request label Apr 12, 2023
@bohdan-vorona bohdan-vorona self-assigned this Apr 12, 2023
@bohdan-vorona bohdan-vorona linked an issue Apr 12, 2023 that may be closed by this pull request
common/models/Order.php Show resolved Hide resolved
common/models/Subscription.php Outdated Show resolved Hide resolved
common/models/Subscription.php Outdated Show resolved Hide resolved
common/models/Subscription.php Outdated Show resolved Hide resolved
common/models/Subscription.php Outdated Show resolved Hide resolved
* @param integer $id
* @return Subscription the loaded model
* @throws NotFoundHttpException if the model cannot be found
* TODO: implement the logic
Copy link
Collaborator

Choose a reason for hiding this comment

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

???

frontend/controllers/SubscriptionWebhookController.php Outdated Show resolved Hide resolved
frontend/controllers/SubscriptionWebhookController.php Outdated Show resolved Hide resolved
frontend/views/subscription/_stripe-widget.php Outdated Show resolved Hide resolved
frontend/views/subscription/_stripe-widget.php Outdated Show resolved Hide resolved
bohdan-vorona and others added 13 commits April 14, 2023 12:32
@bohdan-vorona bohdan-vorona requested a review from samdark April 17, 2023 10:34
@bohdan-vorona bohdan-vorona requested a review from samdark April 17, 2023 17:12
@@ -35,8 +35,8 @@
],
'stripe' => [
'class' => 'common\components\StripeComponent',
'publishableKey' => $params['stripe']['publishable_key'],
'secretKey' => $params['stripe']['secret_key'],
'publicKey' => $params['stripe']['publishable_key'],
Copy link
Collaborator

@samdark samdark Apr 18, 2023

Choose a reason for hiding this comment

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

Oww... sorry. My bad. These are actually named "secret key" and "publishable key" in Stripe API so it worth naming these the same way in the component.

https://stripe.com/docs/keys

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No problem. I've reverted back my latest changes - e696a17

@bohdan-vorona bohdan-vorona requested a review from samdark April 18, 2023 06:20
@cgsmith
Copy link
Contributor

cgsmith commented Apr 20, 2023

Starting review and testing this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stripe Billing and Invoicing [FEATURE]
3 participants