Merge pull request #768 from specklesystems/dev

Update dev to main

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;

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<ModelCard> Deserialize(string models) => serializer.Deserialize<List<ModelCard>>(models).NotNull();
 
-  public void SaveState()
+  protected void SaveState()
   {
     lock (_models)
     {