Skip to content

Add optional icmTeamId field for GenevaAutomation step - #278

Merged
jonathan34c merged 1 commit into
mainfrom
jonachang/icrmTeam
Jul 30, 2026
Merged

Add optional icmTeamId field for GenevaAutomation step#278
jonathan34c merged 1 commit into
mainfrom
jonachang/icrmTeam

Conversation

@jonathan34c

Copy link
Copy Markdown
Contributor

Add optional icmTeamId field for GenevaAutomation step

Copilot AI review requested due to automatic review settings July 29, 2026 23:39

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds support for specifying an optional ICM team identifier on the PublishGenevaAutomation pipeline step, enabling additional ICM routing/association data to be carried through pipeline definitions.

Changes:

  • Extend the pipeline JSON schema to allow an icmTeamId property on publishGenevaAutomationStep.
  • Extend the Go PublishGenevaAutomationStep type to include IcmTeamId and include it in dependency discovery.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
pipelines/types/pipeline.schema.v1.json Adds icmTeamId to the publishGenevaAutomationStep schema properties.
pipelines/types/common.go Adds IcmTeamId to PublishGenevaAutomationStep and updates RequiredInputs() dependency collection.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pipelines/types/common.go
Copilot AI review requested due to automatic review settings July 29, 2026 23:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@jonathan34c
jonathan34c force-pushed the jonachang/icrmTeam branch 8 times, most recently from 98bebec to 6eab680 Compare July 30, 2026 17:22
@jonathan34c
jonathan34c enabled auto-merge (squash) July 30, 2026 17:32
@jonathan34c
jonathan34c merged commit e8fcd63 into main Jul 30, 2026
2 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.

3 participants