Skip to content
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

feat: Add customer type to services #2551

Merged
merged 4 commits into from
Sep 9, 2024

Conversation

brunomiguelpinto
Copy link
Contributor

@brunomiguelpinto brunomiguelpinto commented Sep 9, 2024

Context

We currently only create Lago customers as companies, but there is a need to support both companies and individuals. This change is motivated by scenarios where customers may be a mix of B2B and B2C, and where external integrations require handling both Contacts and Companies.

To address this, we are introducing a new field, customer_type, to distinguish whether a customer is a company or an individual. Existing customers will remain unaffected with customer_type set to the default nil.

Description

In this PR we're adding the new model fields lastname, firstname and customer_type to create and update customer services

@brunomiguelpinto brunomiguelpinto marked this pull request as draft September 9, 2024 14:59
@brunomiguelpinto brunomiguelpinto self-assigned this Sep 9, 2024
@brunomiguelpinto brunomiguelpinto added the Customer Customer management label Sep 9, 2024
@brunomiguelpinto brunomiguelpinto marked this pull request as ready for review September 9, 2024 15:05
@brunomiguelpinto brunomiguelpinto changed the title feat: added new params feat: Add customer type to services Sep 9, 2024
@brunomiguelpinto brunomiguelpinto merged commit 50056b1 into main Sep 9, 2024
6 checks passed
@brunomiguelpinto brunomiguelpinto deleted the feat-customer-type-controllers-services branch September 9, 2024 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Customer Customer management
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants