Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: afw
-
Labels:
-
Story Points:4
-
Epic Link:
-
Sprint:AP F21-5 (October)
-
Team:Alert Production
-
Urgent?:No
Description
exposureId in VisitInfo is not actually a visit-level value, but a ccd-specific value. This requires deprecating `getExposureId()` in visitInfo, so that we can remove it, and replacing any uses of that method with the replacement from DM-29955.
Attachments
Issue Links
- blocks
-
DM-31706 Do Release 24.0.0 of Science Pipelines
- Done
-
DM-32138 Remove VisitInfo.exposureId
- Done
- is blocked by
-
DM-29955 Add ExposureInfo id getter (replacement for VisitInfo.getExposureId())
- Done
-
DM-13944 add id to VisitInfo
- Done
- is duplicated by
-
DM-13738 Remove exposureId from visitInfo
- Invalid
- is triggered by
-
RFC-459 Remove exposureId from VisitInfo and add visitId
- Implemented
Activity
Field | Original Value | New Value |
---|---|---|
Epic Link |
|
Summary | Move exposureId from VisitInfo to ExposureIfno | Move exposureId from VisitInfo to ExposureInfo |
Urgent? | off | |
Labels | SciencePipelines | PairCoding SciencePipelines |
Assignee | John Parejko [ parejkoj ] |
Labels | PairCoding SciencePipelines | PairCoding |
Team | Alert Production [ 10300 ] |
Rank | Ranked higher |
Rank | Ranked higher |
Rank | Ranked higher |
Rank | Ranked higher |
Description | {{exposureId}} in {{VisitInfo}} is not actually a visit-level value, but a ccd-specific value. Move it out of {{VisitInfo}} and into {{ExposureInfo}}, so it lives in the correct place. |
{{exposureId}} in {{VisitInfo}} is not actually a visit-level value, but a ccd-specific value. Move it out of {{VisitInfo}} and into {{ExposureInfo}}, so it lives in the correct place.
Note: this requires deprecating `getExposureId()` in visitInfo. |
Sprint | AP S21-4 (March) [ 1079 ] |
Rank | Ranked lower |
Story Points | 4 |
Sprint | AP S21-4 (March) [ 1079 ] | AP S21-5 (April) [ 1084 ] |
Rank | Ranked lower |
Sprint | AP S21-5 (April) [ 1084 ] | AP S21-6 (May) [ 1088 ] |
Rank | Ranked lower |
Summary | Move exposureId from VisitInfo to ExposureInfo | Deprecate VisitInfo.getExposureId() |
Description |
{{exposureId}} in {{VisitInfo}} is not actually a visit-level value, but a ccd-specific value. Move it out of {{VisitInfo}} and into {{ExposureInfo}}, so it lives in the correct place.
Note: this requires deprecating `getExposureId()` in visitInfo. |
{{exposureId}} in {{VisitInfo}} is not actually a visit-level value, but a ccd-specific value. This requires deprecating `getExposureId()` in visitInfo, so that we can remove it, and replacing any uses of that method with the replacement from |
Sprint | AP S21-6 (May) [ 1088 ] |
Rank | Ranked higher |
Rank | Ranked higher |
Rank | Ranked higher |
Rank | Ranked higher |
Sprint | AP F21-3 (August) [ 1109 ] |
Labels | PairCoding | PairCoding ap-analysis |
Assignee | John Parejko [ parejkoj ] | Krzysztof Findeisen [ krzys ] |
Sprint | AP F21-3 (August) [ 1109 ] | AP F21-2 (July) [ 1102 ] |
Rank | Ranked lower |
Sprint | AP F21-2 (July) [ 1102 ] | AP F21-3 (August) [ 1109 ] |
Rank | Ranked higher |
Sprint | AP F21-3 (August) [ 1109 ] | AP F21-4 (September) [ 1113 ] |
Sprint | AP F21-4 (September) [ 1113 ] | AP F21-5 (October) [ 1119 ] |
Rank | Ranked lower |
Rank | Ranked higher |
Sprint | AP F21-5 (October) [ 1119 ] | AP F21-6 (November) [ 1124 ] |
Rank | Ranked lower |
Status | To Do [ 10001 ] | In Progress [ 3 ] |
Status | In Progress [ 3 ] | In Review [ 10004 ] |
Reviewers | Spencer Nelson [ swnelson ] |
Status | In Review [ 10004 ] | Reviewed [ 10101 ] |
Resolution | Done [ 10000 ] | |
Status | Reviewed [ 10101 ] | Done [ 10002 ] |
Sprint | AP F21-6 (November) [ 1124 ] | AP F21-5 (October) [ 1119 ] |
From DM with John Parejko: