maybe icon indicating copy to clipboard operation
maybe copied to clipboard

Published 20 hours ago • verified publisher icon maybe-finance

Remove billing & upgrade references when STRIPE_API_KEY is missing

Open simonbukin opened this issue 1 year ago • 2 comments

This PR takes a stab at removing upgrade and billing references when STRIPE_API_KEY is not set in the .env file. #151

I'm unsure if there are other places that reference billing and upgrade, but I would be happy to update those as well. Just spun up this repo this morning, so I'm unsure if there are other places these are mentioned.

simonbukin avatar Jan 18 '24 22:01 simonbukin

There's also an email that's sent on a scheduled job for trial upgrade reminders in email.processor.ts, we can noop that in main.ts in the workers folder if no api key.

tmyracle avatar Jan 18 '24 23:01 tmyracle

@tmyracle done!

simonbukin avatar Jan 19 '24 00:01 simonbukin

Thanks @simonbukin!

Shpigford avatar Jan 19 '24 13:01 Shpigford

@simonbukin and @Shpigford there is still a strip container in the docker-compose.yml . Should this be removed as well?

cmosguy avatar Jan 20 '24 17:01 cmosguy

@simonbukin and @Shpigford there is still a strip container in the docker-compose.yml . Should this be removed as well?

No, as we'll still be making use of Stripe for our hosted solution. This PR is just about hiding UI elements.

Shpigford avatar Jan 20 '24 17:01 Shpigford

Ah understood - ok thank you!

cmosguy avatar Jan 20 '24 18:01 cmosguy