Skip to content

feat(integrations): Return externalId from OrganizationIntegrationReposEndpoint#112529

Merged
wedamija merged 2 commits intomasterfrom
danf/org-integration-repos-external-id
Apr 8, 2026
Merged

feat(integrations): Return externalId from OrganizationIntegrationReposEndpoint#112529
wedamija merged 2 commits intomasterfrom
danf/org-integration-repos-external-id

Conversation

@wedamija
Copy link
Copy Markdown
Member

@wedamija wedamija commented Apr 8, 2026

As requested in #112327 (comment)

@wedamija wedamija requested a review from a team April 8, 2026 20:55
@wedamija wedamija requested review from a team as code owners April 8, 2026 20:55
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 8, 2026
Copy link
Copy Markdown
Member

@jaydgoss jaydgoss left a comment

Choose a reason for hiding this comment

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

TY

The new externalId field in OrganizationIntegrationReposEndpoint
requires external_id in the repository data. The acceptance test
mocks were missing this field, causing KeyError → 500 errors.
@wedamija wedamija enabled auto-merge (squash) April 8, 2026 22:11
@wedamija wedamija merged commit 275aa80 into master Apr 8, 2026
56 checks passed
@wedamija wedamija deleted the danf/org-integration-repos-external-id branch April 8, 2026 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants