Skip to content

fix: connect git integration payload (CM-756)#3544

Merged
mbani01 merged 2 commits into
mainfrom
fix/connect_git_integration_payload
Oct 24, 2025
Merged

fix: connect git integration payload (CM-756)#3544
mbani01 merged 2 commits into
mainfrom
fix/connect_git_integration_payload

Conversation

@mbani01
Copy link
Copy Markdown
Contributor

@mbani01 mbani01 commented Oct 24, 2025

This pull request updates the way remote repository data is handled during Git integration authentication. The main change is to ensure that the remotes field in the integration payload is always an array of objects with a url and a forkedFrom property, rather than just an array of URLs.

Integration payload improvements:

  • In gitAuthenticate.ts, the remotes field in the payload sent to IntegrationService.gitConnectOrUpdate is now transformed into an array of objects with url and forkedFrom: null properties, ensuring consistent structure for downstream processing.

@mbani01 mbani01 self-assigned this Oct 24, 2025
@github-actions
Copy link
Copy Markdown
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@mbani01 mbani01 changed the title fix(git-integration): connect git integration payload fix(git-integration): connect git integration payload [CM-756] Oct 24, 2025
@mbani01 mbani01 changed the title fix(git-integration): connect git integration payload [CM-756] fix: connect git integration payload (CM-756) Oct 24, 2025
@mbani01 mbani01 merged commit e662dc2 into main Oct 24, 2025
15 checks passed
@mbani01 mbani01 deleted the fix/connect_git_integration_payload branch October 24, 2025 11:11
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