Skip to content

Conversation

@groyoh
Copy link
Contributor

@groyoh groyoh commented Oct 7, 2025

Context

Currently, our customers cannot sync invoices to multiple accounting entities (subsidiaries) as our mapping is made at the organization level. However, in NetSuite, for instance, the tax and item mapping are made at the entity level. We need to allow our mapping to be defined at the entity level.

Description

This is the first PR where we add a billing_entity_id field to the integration_collection_mappings along with the proper indexes and validations.

There will be two follow-up PR:

  1. A PR which update the integration_mappings similarly.
  2. A PR which update the integration payload services to define a lookup and fallback strategy based on the available mappings.

@groyoh groyoh force-pushed the feat/entity-integration-collection-mapping branch from 1ace3ec to e0276ba Compare October 7, 2025 06:57
@groyoh groyoh marked this pull request as ready for review October 7, 2025 07:04
@groyoh groyoh requested a review from julienbourdeau October 7, 2025 07:04
@groyoh groyoh force-pushed the feat/entity-integration-collection-mapping branch 2 times, most recently from fe52a1d to aba2eec Compare October 7, 2025 07:39
@groyoh groyoh changed the title feat(integration): Add billing_entity_id to integration mapping collection feat(integration): Add billing_entity_id to integration collection mapping Oct 7, 2025
@groyoh groyoh force-pushed the feat/entity-integration-collection-mapping branch from aba2eec to a49b5c3 Compare October 7, 2025 08:38
@groyoh
Copy link
Contributor Author

groyoh commented Oct 8, 2025

Thanks for the review @julienbourdeau 🙏

@groyoh groyoh force-pushed the feat/entity-integration-collection-mapping branch from 129717b to 174d7c4 Compare October 9, 2025 08:38
@groyoh groyoh force-pushed the feat/entity-integration-collection-mapping branch from 174d7c4 to 1c11988 Compare October 29, 2025 16:45
@groyoh groyoh force-pushed the feat/entity-integration-collection-mapping branch from 1c11988 to a46d524 Compare October 31, 2025 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants