pix3lot / Slackit

Linking Slack with Revit
28 stars 7 forks source link

Get _start_state on DocOpen, regardless of settings #14

Closed djsiroky closed 7 years ago

djsiroky commented 7 years ago

This should resolve #13, where someone opens a project and doesn't have pinned element tracking enabled, but then enables it and syncs. Since it previously didn't grab _start_state, the ReportChanges method would then fail due to a null _start_state.

I explored options of getting the _start_state when someone changes their SlackIt settings, but this would also have to account for receiving new changes via syncing the model. You'd also have to track the before and after save settings to see if it is enabled, which would add a layer of complexity. _start_state is already being updated every time you sync, so also doing so on document open is more in line with that pattern.