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 a follow-up PR to #4445 and #4446.

This makes sure that each integration rely on the correct integration mapping or integration collection mapping following this order:

  1. Mapping for item on billing entity
  2. Fallback item on billing entity
  3. Mapping for item on organization
  4. Fallback item on organization

To make sure the behavior is correctly handled on all integrations, I'll heavily update the tests in an other PR.

@groyoh groyoh force-pushed the feat/entity-integration-payload branch from 63feea3 to ea18887 Compare October 7, 2025 14:11
@groyoh groyoh force-pushed the feat/entity-integration-mapping branch from ef4bf5a to a7bb294 Compare October 8, 2025 09:42
@groyoh groyoh force-pushed the feat/entity-integration-payload branch 2 times, most recently from 6965ce6 to 6e544ef Compare October 9, 2025 08:29
@groyoh groyoh marked this pull request as ready for review October 9, 2025 08:34
@groyoh groyoh force-pushed the feat/entity-integration-mapping branch from a7bb294 to d7ea4ee Compare October 9, 2025 08:38
@groyoh groyoh force-pushed the feat/entity-integration-payload branch 2 times, most recently from f66b227 to 395e62c Compare October 9, 2025 08:43
@groyoh groyoh requested a review from ivannovosad October 9, 2025 09:12
@groyoh
Copy link
Contributor Author

groyoh commented Oct 9, 2025

Thanks for the review @ivannovosad 🙏

@groyoh groyoh force-pushed the feat/entity-integration-payload branch from 395e62c to ebe1186 Compare October 29, 2025 16:46
@groyoh groyoh force-pushed the feat/entity-integration-mapping branch from d7ea4ee to cd1999e Compare October 31, 2025 08:58
@groyoh groyoh force-pushed the feat/entity-integration-payload branch from ebe1186 to d1d5639 Compare October 31, 2025 08:59
@groyoh groyoh force-pushed the feat/entity-integration-mapping branch from cd1999e to 55052ce Compare October 31, 2025 09:25
@groyoh groyoh force-pushed the feat/entity-integration-payload branch from d1d5639 to 17c5de4 Compare October 31, 2025 09:26
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.

4 participants