Revert "feat: allow updates and deletes on time-series collections COMPASS-6786" (#4396)

Revert "feat: allow updates and deletes on time-series collections COMPASS-6786 (#4382)"

This reverts commit 411feec.

Author: Anemy

package-lock.json

@@ -105,7 +105,6 @@
     "react-dom": "^17.0.2",
     "reflux": "^0.4.1",
     "reflux-state-mixin": "github:mongodb-js/reflux-state-mixin",
-    "semver": "^5.4.1",
     "sinon": "^8.1.1",
     "webpack": "^4.46.0"
   },

packages/compass-crud/package.json

@@ -27,7 +27,6 @@ export type DocumentJsonViewProps = {
 } & Pick<
   JSONEditorProps,
   | 'isTimeSeries'
-  | 'serverVersion'
   | 'copyToClipboard'
   | 'removeDocument'
   | 'replaceDocument'
@@ -62,7 +61,6 @@ class DocumentJsonView extends React.Component<DocumentJsonViewProps> {
               doc={doc}
               editable={this.props.isEditable}
               isTimeSeries={this.props.isTimeSeries}
-              serverVersion={this.props.serverVersion}
               copyToClipboard={this.props.copyToClipboard}
               removeDocument={this.props.removeDocument}
               replaceDocument={this.props.replaceDocument}

packages/compass-crud/src/components/document-json-view.tsx

@@ -29,7 +29,6 @@ export type DocumentListViewProps = {
 } & Pick<
   DocumentProps,
   | 'isTimeSeries'
-  | 'serverVersion'
   | 'copyToClipboard'
   | 'removeDocument'
   | 'replaceDocument'
@@ -62,7 +61,6 @@ class DocumentListView extends React.Component<DocumentListViewProps> {
               doc={doc}
               editable={this.props.isEditable}
               isTimeSeries={this.props.isTimeSeries}
-              serverVersion={this.props.serverVersion}
               copyToClipboard={this.props.copyToClipboard}
               removeDocument={this.props.removeDocument}
               replaceDocument={this.props.replaceDocument}
@@ -93,7 +91,6 @@ class DocumentListView extends React.Component<DocumentListViewProps> {
     docs: PropTypes.array.isRequired,
     isEditable: PropTypes.bool,
     isTimeSeries: PropTypes.bool,
-    serverVersion: PropTypes.string,
     removeDocument: PropTypes.func,
     replaceDocument: PropTypes.func,
     updateDocument: PropTypes.func,

packages/compass-crud/src/components/document-list-view.tsx

@@ -343,7 +343,6 @@ class DocumentList extends React.Component<DocumentListProps> {
     isEditable: PropTypes.bool.isRequired,
     isExportable: PropTypes.bool.isRequired,
     isTimeSeries: PropTypes.bool,
-    serverVersion: PropTypes.string,
     store: PropTypes.object.isRequired,
     openInsertDocumentDialog: PropTypes.func,
     openImportFileDialog: PropTypes.func,
@@ -398,7 +397,6 @@ DocumentList.propTypes = {
   isEditable: PropTypes.bool.isRequired,
   isExportable: PropTypes.bool.isRequired,
   isTimeSeries: PropTypes.bool,
-  serverVersion: PropTypes.string,
   store: PropTypes.object.isRequired,
   openInsertDocumentDialog: PropTypes.func,
   openImportFileDialog: PropTypes.func,

packages/compass-crud/src/components/document-list.tsx

@@ -6,13 +6,11 @@ import EditableDocument from './editable-document';
 import type { ReadonlyDocumentProps } from './readonly-document';
 import ReadonlyDocument from './readonly-document';
 import type { BSONObject } from '../stores/crud-store';
-import { hasEditableTimeSeriesSupport } from '../utils/editable-time-series';
 
 export type DocumentProps = {
   doc: HadronDocument | BSONObject;
   editable: boolean;
   isTimeSeries?: boolean;
-  serverVersion?: string;
   isExpanded?: boolean;
 } & Omit<EditableDocumentProps, 'doc' | 'expandAll'> &
   Pick<ReadonlyDocumentProps, 'copyToClipboard' | 'openInsertDocumentDialog'>;
@@ -21,7 +19,6 @@ const Document = (props: DocumentProps) => {
   const {
     editable,
     isTimeSeries,
-    serverVersion,
     isExpanded,
     copyToClipboard,
     openInsertDocumentDialog,
@@ -37,11 +34,7 @@ const Document = (props: DocumentProps) => {
     return new HadronDocument(_doc as any);
   }, [_doc]);
 
-  if (
-    editable &&
-    isTimeSeries &&
-    !hasEditableTimeSeriesSupport(serverVersion)
-  ) {
+  if (editable && isTimeSeries) {
     return (
       <ReadonlyDocument
         doc={doc}
@@ -69,7 +62,6 @@ Document.propTypes = {
   doc: PropTypes.object.isRequired,
   editable: PropTypes.bool,
   isTimeSeries: PropTypes.bool,
-  serverVersion: PropTypes.string,
   removeDocument: PropTypes.func,
   replaceDocument: PropTypes.func,
   updateDocument: PropTypes.func,

packages/compass-crud/src/components/document.tsx

@@ -52,7 +52,6 @@ export type JSONEditorProps = {
   doc: Document;
   editable: boolean;
   isTimeSeries?: boolean;
-  serverVersion?: string;
   removeDocument?: CrudActions['removeDocument'];
   replaceDocument?: CrudActions['replaceDocument'];
   updateDocument?: CrudActions['updateDocument'];

packages/compass-crud/src/components/json-editor.tsx

@@ -206,7 +206,6 @@ describe('store', function () {
         isEditable: true,
         isReadonly: false,
         isTimeSeries: false,
-        serverVersion: '4.0.0',
         isWritable: false,
         ns: '',
         outdated: false,

packages/compass-crud/src/stores/crud-store.spec.ts

@@ -190,19 +190,6 @@ export const setIsTimeSeries = (
   store.onTimeSeriesChanged(isTimeSeries);
 };
 
-/**
- * Set the serverVersion flag in the store.
- *
- * @param {Store} store - The store.
- * @param {Boolean} serverVersion - The current version of the server.
- */
-export const setServerVersion = (
-  store: CrudStoreImpl,
-  serverVersion: string
-) => {
-  store.onServerVersionChanged(serverVersion);
-};
-
 /**
  * Set the namespace in the store.
  *
@@ -268,7 +255,6 @@ type CrudStoreOptions = {
   isReadonly: boolean;
   namespace: string;
   isTimeSeries: boolean;
-  serverVersion: string;
   dataProvider: { error?: Error; dataProvider?: DataService };
   noRefreshOnConfigure?: boolean;
 };
@@ -333,7 +319,6 @@ type CrudState = {
   isDataLake: boolean;
   isReadonly: boolean;
   isTimeSeries: boolean;
-  serverVersion: string;
   status: DOCUMENTS_STATUSES;
   debouncingLoad: boolean;
   loadingCount: boolean;
@@ -389,7 +374,6 @@ class CrudStoreImpl
       isDataLake: false,
       isReadonly: false,
       isTimeSeries: false,
-      serverVersion: '4.0.0',
       status: DOCUMENTS_STATUS_INITIAL,
       debouncingLoad: false,
       loadingCount: false,
@@ -488,15 +472,6 @@ class CrudStoreImpl
     this.setState({ isTimeSeries });
   }
 
-  /**
-   * Set the current server version.
-   *
-   * @param {Boolean} serverVersion - The current server version.
-   */
-  onServerVersionChanged(serverVersion: string) {
-    this.setState({ serverVersion });
-  }
-
   /**
    * Plugin lifecycle method that is called when the namespace changes in
    * Compass. Trigger with new namespace and cleared path/types.
@@ -1563,10 +1538,6 @@ const configureStore = (options: CrudStoreOptions & GridStoreOptions) => {
     setIsTimeSeries(store, options.isTimeSeries);
   }
 
-  if (options.serverVersion) {
-    setServerVersion(store, options.serverVersion);
-  }
-
   if (options.dataProvider) {
     setDataProvider(
       store,