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

Check alert schemas for validity

    Details

    • Story Points:
      4
    • Sprint:
      AP S20-4 (March)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      During the review of https://jira.lsstcorp.org/browse/DM-23588, John Swinbank produced some additional test code at https://github.com/lsst-dm/sample-avro-alert/pull/10. This ticket is to review the code, expand it (or its tests) as you view necessary, and merge it to master.

        Attachments

          Issue Links

            Activity

            Hide
            swinbank John Swinbank added a comment -

            Hey Spencer — Thanks for your comments on my earlier PR. I made some changes and put up a new PR here: https://github.com/lsst-dm/sample-avro-alert/pull/11. Please could you take another look?

            (New PR was necessary due to changing ticket number in our workflow; sorry that it makes it harder to refer back to your earlier comments.)

            Show
            swinbank John Swinbank added a comment - Hey Spencer — Thanks for your comments on my earlier PR. I made some changes and put up a new PR here: https://github.com/lsst-dm/sample-avro-alert/pull/11 . Please could you take another look? (New PR was necessary due to changing ticket number in our workflow; sorry that it makes it harder to refer back to your earlier comments.)
            Hide
            swinbank John Swinbank added a comment -

            Merged & done.

            Show
            swinbank John Swinbank added a comment - Merged & done.

              People

              • Assignee:
                swinbank John Swinbank
                Reporter:
                ebellm Eric Bellm
                Reviewers:
                Spencer Nelson
                Watchers:
                Eric Bellm, John Swinbank, Spencer Nelson
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel