1. 22 Feb, 2021 2 commits
  2. 20 Feb, 2021 2 commits
    • Kerri Miller's avatar
      Move #mark_as_checked last and add comment · 20896b28
      Kerri Miller authored
      20896b28
    • Kerri Miller's avatar
      Refactor tracking of title, desc, and draft status changes · d8acf44c
      Kerri Miller authored
      When I initially added the tracking logic here, I hadn't been able to
      track down where we were triggering additional update events, which was
      resetting ActiveModel::Dirty#changes (etc) pushing me to cobble together
      holding state of incoming changes. This was a fine approach, but it
      avoided tackling the complexity of this service head-on. In returning to
      the code today, I see now how #mark_as_unchecked is triggering an update
      on state change (thanks to the state machine behind that..) and can
      sequence the code more appropriately and, critically, more flexibly for
      deliverables I have in this release.
      d8acf44c
  3. 19 Feb, 2021 36 commits