-
-
Notifications
You must be signed in to change notification settings - Fork 716
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
Prefix HOBBY_SUBSCRIPTION_PRICE_ID, PRO_SUBSCRIPTION_PRICE_ID and CREDITS_PRICE_ID env vars with STRIPE_ ? #145
Comments
I am also getting errors here about which ID to put here? Also onclick my page isnt redirecting to stripe payment page |
Hi @rugs07 have you setup stripe as explained here https://docs.opensaas.sh/guides/stripe-integration ? If so, can you share the console log statements coming from the server? |
yeah these are specific to Stripe and should be prefixed with |
Sure vincanger,The console log statements are as below |
@rugs07 the error is telling you that stripe tax is not activated on your account. I think for some countries this is necessary or you can deactivate it in the return await stripe.checkout.sessions.create({
line_items: [
{
price: priceId,
quantity: 1,
},
],
mode: mode,
success_url: `${DOMAIN}/checkout?success=true`,
cancel_url: `${DOMAIN}/checkout?canceled=true`,
automatic_tax: { enabled: false }, // changed to FALSE
customer_update: {
address: 'auto',
},
customer: customerId,
});
``` |
Also what about this part? |
@rugs07 it might be useful if you go to our Discord Server and ask this question there in #questions, it is a better place for helping with using OpenSaas, and other community members are more likely to help, while here it is more about technical aspects of open saas itself. |
PR --> #183 |
These are in .env.server, they are next to STRIPE_ env vars, but are not prefixed with STRIPE_. Why is that so? Are these not Stripe specific? Because of
id
part, it sounds to me like that are specific for Stripe, thatid
is id on Stripe.If they are not specific for Stripe then ok, but if they are, we should prefix them also right?
The text was updated successfully, but these errors were encountered: