Skip to content

Onboarding - delete PENDING_CREATION workspace if billing is deactivated #12704

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

Merged
merged 2 commits into from
Jun 20, 2025

Conversation

etiennejouan
Copy link
Contributor

@etiennejouan etiennejouan commented Jun 18, 2025

@etiennejouan etiennejouan changed the title Onboarding - delete PENDING_CREATION workspace if billing is deactivated DRAFT - Onboarding - delete PENDING_CREATION workspace if billing is deactivated Jun 18, 2025
@etiennejouan etiennejouan force-pushed the ej/fix-pending-creation branch from f1c23d1 to f80698a Compare June 20, 2025 15:35
@etiennejouan etiennejouan changed the title DRAFT - Onboarding - delete PENDING_CREATION workspace if billing is deactivated Onboarding - delete PENDING_CREATION workspace if billing is deactivated Jun 20, 2025
@etiennejouan etiennejouan marked this pull request as ready for review June 20, 2025 15:37
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Refactors billing webhook functionality into a dedicated billing-webhook core module, adding workspace cleanup for deactivated billing states. Key architectural improvements include:

  • Extracted billing webhook handling from billing module into dedicated billing-webhook module to improve separation of concerns
  • Added logic to delete workspaces in PENDING_CREATION state when billing is deactivated via webhook
  • Consolidated all Stripe event transformers and webhook services under the new module while maintaining existing functionality
  • Improved webhook controller error handling with specific BillingException cases
  • Streamlined module organization by properly grouping related webhook services (alerts, invoices, subscriptions, etc.)

26 files reviewed, 1 comment
Edit PR Review Bot Settings | Greptile

Copy link
Contributor

github-actions bot commented Jun 20, 2025

🚀 Preview Environment Ready!

Your preview environment is available at: http://bore.pub:62086

This environment will automatically shut down when the PR is closed or after 5 hours.

@etiennejouan etiennejouan merged commit 830e49c into main Jun 20, 2025
46 checks passed
@etiennejouan etiennejouan deleted the ej/fix-pending-creation branch June 20, 2025 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants