jenkinsci / jenkinsci/branch-api-plugin

[JENKINS-48564] SCMSource.id should be part of out of the box API jelly

Open
#689 1 comment 0 reactions 0 assignees View on GitHub
component:branch-api-plugin imported-jira-issue priority:minor resolution:unresolved
Dominant language
Java
Stars
39
Forks
154
PR merge metrics
No merged PRs in 30d

Description

Looks like after some of refactorings SCMSource.id become bundled in SCMSource that has DataBound annotations, but nowhere mentioned that implementators should insert invisible fields for scmsource, either it's id will be constantly changing that influence on a lot of places.

branch-api is already wrapping config-details, so it should handle `id` itself out of the box.

---
Originally reported by integer, imported from: SCMSource.id should be part of out of the box API jelly


  • status: Open
  • priority: Minor
  • component(s): branch-api-plugin
  • resolution: Unresolved
  • votes: 0
  • watchers: 2
  • imported: 20251211-141027

Raw content of original issue

Looks like after some of refactorings SCMSource.id become bundled in SCMSource that has DataBound annotations, but nowhere mentioned that implementators should insert invisible fields for scmsource, either it's id will be constantly changing that influence on a lot of places.
branch-api is already wrapping config-details, so it should handle `id` itself out of the box.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.