Skip to content

Fix for PM-6579 - #55

Merged
jmgasper merged 2 commits into
masterfrom
dev
Oct 16, 2026
Merged

jmgasper merged 2 commits into
masterfrom
dev

Conversation

@jmgasper

Copy link
Copy Markdown
Contributor

No description provided.

What was broken

Deal details omitted the Work project and billing history for extensions whose
billing account differs from the shared project's current account.

Root cause

The opportunity endpoint only matched Project.billingAccountId and ignored the
explicit Salesforce Work/Connect project reference already used by Sales reports.

What was changed

Read TC_Connect_Project_Id__c and use it to resolve a non-deleted project when
no current billing-account match exists. Validate the reference, preserve the
opportunity's own account, and reuse the existing project billing history lookup.
Update the service documentation, response DTO description, and README.

Any added/updated tests

Added 17 regression cases covering shared extension projects, account-match
precedence, invalid/missing/deleted references, and lookup failures. The extension
regressions fail against the previous implementation. All 56 focused API tests
and all 6 existing Sales modal tests pass. pnpm lint and pnpm build pass.

Full unit and e2e suites were run. Their 15 unit and 8 e2e failures reproduce
unchanged on origin/dev (metadata events, JWT handling, event fixtures, and
permission-system checks); this change adds no failures.
PM-6579: Resolve shared Work projects for deal extensions
@jmgasper
jmgasper merged commit 93b58a7 into master Oct 16, 2026
5 checks passed
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.

1 participant