Stripe pricing table again #420
Conversation
✅ Deploy Preview for storied-crostata-fa92db ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for dreamy-paletas-cb5e15 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
| @@ -18,92 +18,24 @@ | |||
| </div> | |||
|
|
|||
| <div class="row"> | |||
| <div class="small-12 columns"> | |||
There was a problem hiding this comment.
❓ Does removing this class do anything?
There was a problem hiding this comment.
I don't remember why I removed this specifically (likely faffing around with the pricing table width), though small-12 is just width: 100%, so I think it only makes sense if you're setting different widths at different breakpoints, right? Otherwise the div's width is always 100%.
| <script async src="https://js.stripe.com/v3/pricing-table.js"></script> | ||
| <stripe-pricing-table pricing-table-id="prctbl_1PYSAdHVcZRgX183ab67RI0g" | ||
| publishable-key="pk_live_1sIaEOF4fdxq136517FMhqG0"> | ||
| </stripe-pricing-table> |
There was a problem hiding this comment.
NAB: Do we need to do anything in case the Pricing Table doesn't load (for whatever reason)?
There was a problem hiding this comment.
Yeah, good question. I didn't see anything in the Stripe documentation about it, so...no? It loads a custom element so we could check with something like
const pricingTableLoaded = !!customElements.get('stripe-pricing-table');
if (!pricingTableLoaded) {
// insert fallback
}
🚨 Don't merge this 😄 🚨
⚡ THIS IS A LIVE TABLE ⚡
If you purchase through this you will be charged actual real money. If you're keen to test, there's a discount code, which you can ask for in Slack.