diff --git a/Connectors/Revit/Speckle.Connectors.RevitShared/Bindings/RevitSendBinding.cs b/Connectors/Revit/Speckle.Connectors.RevitShared/Bindings/RevitSendBinding.cs index 1d40afda6..f87a4f2c0 100644 --- a/Connectors/Revit/Speckle.Connectors.RevitShared/Bindings/RevitSendBinding.cs +++ b/Connectors/Revit/Speckle.Connectors.RevitShared/Bindings/RevitSendBinding.cs @@ -263,10 +263,6 @@ internal sealed class RevitSendBinding : RevitBaseBinding, ISendBinding modelCard.SendFilter.IdMap, newSelectedObjectIds ); - - // In UI, we are not explicitly trigger UpdateModel to save the state into file for some edge cases. Instead we save it directly via document model store. - // Comment in the UI -> """ otherwise it is leading cleanup on document store bc of deferred action when we switched to the another doc""" - _store.SaveState(); } return documentElementContexts; diff --git a/DUI3/Speckle.Connectors.DUI/Models/DocumentModelStore.cs b/DUI3/Speckle.Connectors.DUI/Models/DocumentModelStore.cs index a4e0c7c09..dc82c2b61 100644 --- a/DUI3/Speckle.Connectors.DUI/Models/DocumentModelStore.cs +++ b/DUI3/Speckle.Connectors.DUI/Models/DocumentModelStore.cs @@ -133,7 +133,7 @@ public abstract class DocumentModelStore(IJsonSerializer serializer) // POC: this seemms more like a IModelsDeserializer?, seems disconnected from this class protected List Deserialize(string models) => serializer.Deserialize>(models).NotNull(); - public void SaveState() + protected void SaveState() { lock (_models) {