Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-34771

Rename existing salindex fields in ts_xml and update unit tests to prohibit salIndex as a field name

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ts_middleware
    • Labels:
    • Story Points:
      0
    • Epic Link:
    • Sprint:
      TSSW Sprint - May 09 - May 23
    • Team:
      Telescope and Site
    • Urgent?:
      No

      Description

      The ts_xml package has one component that uses fields named salIndex: ScriptQueue. Change this field to scriptSalIndex. Also update the unit tests to make "salIndex" a forbidden field name.

        Attachments

          Issue Links

          is child task of

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. CAP-878 Catch-all for T&S tickets for ts_xml 12

          • Undefined - No priority assigned (may not apply or be unknown)
          • Done
          is triggered by

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-849 Rename the {Name}ID field in indexed SAL topics to salIndex

          • Implemented

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            rowen Russell Owen created issue -
            rowen Russell Owen made changes -
            Field Original Value New Value
            Link This issue is triggered by RFC-849 [ RFC-849 ]
            rowen Russell Owen made changes -
            Link This issue is triggering CAP-878 [ CAP-878 ]
            rowen Russell Owen made changes -
            Link This issue is child task of CAP-878 [ CAP-878 ]
            rowen Russell Owen made changes -
            Link This issue is triggering CAP-878 [ CAP-878 ]
            rowen Russell Owen made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            Hide
            Permalink
            rowen Russell Owen added a comment -

            Pull request: https://github.com/lsst-ts/ts_xml/pull/580

            Show
            rowen Russell Owen added a comment - Pull request: https://github.com/lsst-ts/ts_xml/pull/580
            rowen Russell Owen made changes -
            Resolution Done [ 10000 ]
            Status In Progress [ 3 ] Done [ 10002 ]
            Hide
            Permalink
            rowen Russell Owen added a comment -

            Reviewed on github

            Show
            rowen Russell Owen added a comment - Reviewed on github
            Hide
            Permalink
            rowen Russell Owen added a comment -

            Merged to develop

            Show
            rowen Russell Owen added a comment - Merged to develop

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Watchers:
              Russell Owen
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.