jenkinsci / jenkinsci/branch-api-plugin
[JENKINS-48564] SCMSource.id should be part of out of the box API jelly
- 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
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
Assessment
This issue has not been assessed yet.