Skip to content

Remove SessionSummary from SessionState #187

@connor4312

Description

@connor4312

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.

Metadata

Metadata

Assignees

Labels

debtCode quality issues

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions