#805 #784 As a Subscriber I can settle a negative balance myself

[chrisjsimpson]

Ref #805

What's new?

• New Subscriber route http://127.0.0.1:5000/account/failed-invoices As a Subscriber, I have a balance which may fall negative if I fail to pay a subscription after all retry attempts have failed #805
• New Subscriber route: http://127.0.0.1:5000/account/pay-invoice/in_ As a Subscriber I when I login to my account, then I can add money to my subscripton to settle an negative balance #784

How to test

1. As a Subscriber, subscribe and find and use test card which fails payments after inital card attachment.Take note of the failed invoice state open (As shop admin)

2. As a Subscriber, login to the Subscriber login page

3. As a Subscriber, locate and visit the "Failed Invoices" page:

5. As a Subscriber Click 'Pay now' on the failed invoices page
   (failed invoices page)

6. Obvserve: You are redirected to Stripe invoice payment page 7. Use Stripe test card (e.g. the 4242 ... card) to complete the payment 8. Navigate back to shop (is this automatic?) , TODO: refresh invoices, observe the invoice is changed from status `open` to `paid`.

Details

• New concept: "failed invoices"
• new database table stripe_invoice Purpose: To reduce round trip time fetching invoice information from Stripe each time (cache). This is a fast cache (will not be up to date) until refreshed via a get via Stripe api. see detail
  • This is a cache which must be refreshed (either manually or automatically) to fetch Stripe invoices (reglardless of if they are failed or not- all invoices are cached using get_stripe_invoices) into the table stripe_invoice
  • Failed Stripe invoices are fetched via get_stripe_failed_subscription_invoices, which is fast because it uses the cached table stripe_invoice which stores stripe invoices. To determine a failed invoice (from Stripe's perspective is not simple: see detail)
• new database table balance
• new route guard @stripe_connect_id_required, purpose: see comment- does note require stripe connect is activated, only that a stripe connect id exists (and not needed for all pages)
• New admin route /invoices/failed/ to show failed invoices, implemented by get_stripe_failed_subscription_invoices

[github-actions]

🙌 Live preview is here: http://805-subscriber-balance-concept.pcpink.co.uk

[github-actions]

🙌 Live preview is here: http://805-subscriber-balance-concept.pcpink.co.uk

[lgtm-com]

This pull request introduces 1 alert and fixes 1 when merging 37fdd1c into 9cd5da8 - view on LGTM.com

new alerts:

• 1 for URL redirection from remote source

fixed alerts:

• 1 for URL redirection from remote source

[github-actions]

🙌 Live preview is here: http://805-subscriber-balance-concept.pcpink.co.uk

[lgtm-com]

This pull request introduces 1 alert and fixes 1 when merging 66935df into 9cd5da8 - view on LGTM.com

new alerts:

• 1 for URL redirection from remote source

fixed alerts:

• 1 for URL redirection from remote source

[github-actions]

🙌 Live preview is here: http://805-subscriber-balance-concept.pcpink.co.uk

[lgtm-com]

This pull request introduces 1 alert and fixes 1 when merging d26e57f into 9cd5da8 - view on LGTM.com

new alerts:

• 1 for URL redirection from remote source

fixed alerts:

• 1 for URL redirection from remote source

[github-actions]

🙌 Live preview is here: http://805-subscriber-balance-concept.pcpink.co.uk

[lgtm-com]

This pull request introduces 1 alert and fixes 1 when merging 8d4754a into 9cd5da8 - view on LGTM.com

new alerts:

• 1 for URL redirection from remote source

fixed alerts:

• 1 for URL redirection from remote source

[github-actions]

🙌 Live preview is here: http://805-subscriber-balance-concept.pcpink.co.uk

[github-actions]

🙌 Live preview is here: http://805-subscriber-balance-concept.pcpink.co.uk

[lgtm-com]

This pull request introduces 1 alert and fixes 1 when merging 9899965 into 9cd5da8 - view on LGTM.com

new alerts:

• 1 for URL redirection from remote source

fixed alerts:

• 1 for URL redirection from remote source

[github-actions]

🙌 Live preview is here: http://805-subscriber-balance-concept.pcpink.co.uk

[lgtm-com]

This pull request introduces 3 alerts and fixes 1 when merging c9ba150 into 6f46253 - view on LGTM.com

new alerts:

• 2 for Unused import
• 1 for URL redirection from remote source

fixed alerts:

• 1 for URL redirection from remote source

[github-actions]

🙌 Live preview is here: http://805-subscriber-balance-concept.pcpink.co.uk

[lgtm-com]

This pull request introduces 3 alerts and fixes 1 when merging fa71b8f into 6f46253 - view on LGTM.com

new alerts:

• 2 for Unused import
• 1 for URL redirection from remote source

fixed alerts:

• 1 for URL redirection from remote source

[github-actions]

🙌 Live preview is here: http://805-subscriber-balance-concept.pcpink.co.uk

[github-actions]

🙌 Live preview is here: http://805-subscriber-balance-concept.pcpink.co.uk

[lgtm-com]

This pull request introduces 3 alerts and fixes 1 when merging c39e626 into 6f46253 - view on LGTM.com

new alerts:

• 2 for Unused import
• 1 for URL redirection from remote source

fixed alerts:

• 1 for URL redirection from remote source

[github-actions]

🙌 Live preview is here: http://805-subscriber-balance-concept.pcpink.co.uk

[github-actions]

🚀 PR was released in v0.1.55 🚀

[lgtm-com]

This pull request introduces 1 alert and fixes 1 when merging c6207ff into 6f46253 - view on LGTM.com

new alerts:

• 1 for URL redirection from remote source

fixed alerts:

• 1 for URL redirection from remote source

[joeltejeda]

#785