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

Add FunctorKey for Boxes

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      DRP S18-2
    • Team:
      Data Release Production

      Description

      We often store Box2Is and Box2Ds in tables. We should have a FunctorKey to do this to reduce code duplication.

      Unfortunately we may not be able to use this actually reduce existing code duplication when code has used a different convention for naming fields (as we'd need to maintain backwards compatibility with already-persisted objects), but this should at least reduce duplication going forward.

      I'm planning to do this now for DM-12370 so I can use it there.

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            John Parejko, could you review this medium-small addition to the suite of afw::table FunctorKeys?

            Show
            jbosch Jim Bosch added a comment - John Parejko , could you review this medium-small addition to the suite of afw::table FunctorKeys?
            Hide
            Parejkoj John Parejko added a comment -

            See comments on PR.

            Show
            Parejkoj John Parejko added a comment - See comments on PR.
            Hide
            jbosch Jim Bosch added a comment -

            Merged to master.

            Show
            jbosch Jim Bosch added a comment - Merged to master.

              People

              • Assignee:
                jbosch Jim Bosch
                Reporter:
                jbosch Jim Bosch
                Reviewers:
                John Parejko
                Watchers:
                Jim Bosch, John Parejko
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel