There are initial debt items around here...
...but while we're here, the whole situation of the SessionSummary being embedded on the SessionState is bad. We have two different sets of actions to update the same thing, which has led to confusion in implementation and makes it really hard to fix #186 nicely.
SessionSummary should share some fields with SessionState, but it should be its own thing that exists solely on the root. In that case the host is just in control of updating the modifiedAt at its own discretion and get we get rid of the oddly overlapping actions.
We can also reduce how much of changesets we want to expose on the summary.
Fix timestamps at the same time.
There are initial debt items around here...
...but while we're here, the whole situation of the SessionSummary being embedded on the SessionState is bad. We have two different sets of actions to update the same thing, which has led to confusion in implementation and makes it really hard to fix #186 nicely.
SessionSummary should share some fields with SessionState, but it should be its own thing that exists solely on the root. In that case the host is just in control of updating the modifiedAt at its own discretion and get we get rid of the oddly overlapping actions.
We can also reduce how much of
changesetswe want to expose on the summary.Fix timestamps at the same time.