Skip to content

feat(jira): support multiple components in project settings (SC-13173)#15039

Open
Maffooch wants to merge 1 commit into
bugfixfrom
feature-13173-jira-multi-component
Open

feat(jira): support multiple components in project settings (SC-13173)#15039
Maffooch wants to merge 1 commit into
bugfixfrom
feature-13173-jira-multi-component

Conversation

@Maffooch

Copy link
Copy Markdown
Contributor

Summary

Implements SC-13173 — allow assigning more than one Jira component from a project's settings.

The JIRA project component field is a single text value that was sent to Jira as one component named after the entire string (e.g. Security,DevSecOps) instead of separate components. Jira's REST API expects a list: [{"name": "Security"}, {"name": "DevSecOps"}].

Changes

  • prepare_jira_issue_fields now splits the component value on commas, trims whitespace, and drops empty entries, producing one Jira component per name. A single value without commas still yields one component.
  • JIRAProjectForm documents that commas separate multiple components (form-level help text; no migration).
  • Updated the Component section in the OS and PRO Jira guides.

Known limitation

A component whose name legitimately contains a comma would be split. This is an accepted trade-off given component names rarely contain commas.

Testing

  • unittests.test_jira_helper.JIRAComponentFieldTest — new (verified red→green): single value, comma split, whitespace trimming, empty-entry dropping, and field omission when no component is set.
  • test_jira_helper → 11 OK, test_jira_config_product → 15 OK.

🤖 Generated with Claude Code

The JIRA project component field is a single text value that was sent to
Jira as one component named after the whole string (e.g. "Security,DevSecOps").
prepare_jira_issue_fields now splits the value on commas (trimming whitespace
and dropping empty entries) so Jira receives a separate component per name.
The project form documents that commas separate multiple components.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@valentijnscholten valentijnscholten left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

what if some one wants to have or is used to be able to use:

Image

@Maffooch

Copy link
Copy Markdown
Contributor Author

what if some one wants to have or is used to be able to use:

Image

That's the tough part about this one... We have to pick a separator, and a comma felt like the best choice for consistency, but I am absolutely open to other options

@valentijnscholten

Copy link
Copy Markdown
Member

Yeah I understand. I think having commas in a component name is not very common. So maybe a note in the upgrade notes is enough.

@mtesauro mtesauro 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.

Approved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants