Fix version vector cleanup during client deactivation (#1068)

This change fixes incorrect version vector persistence during client
deactivation. The issue occurred when an unnecessary UpdateVersionVector call
followed a successful document detachment, re-inserting a version vector that
should have been deleted.

server/clients/clients.go

@@ -87,18 +87,6 @@ func Deactivate(
 		if err := cli.DetachDocument(ctx, project, actorID, docID, project.PublicKey, docInfo.Key); err != nil {
 			return nil, err
 		}
-
-		if err := be.DB.UpdateVersionVector(
-			ctx,
-			info,
-			types.DocRefKey{
-				ProjectID: refKey.ProjectID,
-				DocID:     docID,
-			},
-			nil,
-		); err != nil {
-			return nil, err
-		}
 	}
 
 	return be.DB.DeactivateClient(ctx, refKey)