remove onDocumentSaved support

Signed-off-by: Scott J Dickerson <[email protected]>

Author: sjd78

client/src/app/components/schema-defined-fields/SchemaAsCodeEditor.tsx

@@ -24,7 +24,6 @@ export interface ISchemaAsCodeEditorProps {
   id: string;
   jsonDocument: object;
   jsonSchema?: JsonSchemaObject;
-  onDocumentSaved?: (newSchemaContent: object) => void;
   onDocumentChanged?: (newSchemaContent: object) => void;
   isReadOnly?: boolean;
   /**
@@ -38,7 +37,6 @@ export const SchemaAsCodeEditor = ({
   id,
   jsonDocument,
   jsonSchema,
-  onDocumentSaved,
   onDocumentChanged,
   isReadOnly = false,
   height = "600px",
@@ -48,7 +46,7 @@ export const SchemaAsCodeEditor = ({
   const [currentCode, setCurrentCode] = React.useState(
     JSON.stringify(jsonDocument, null, 2)
   );
-  const [okToSave, setOkToSave] = React.useState(true);
+  // const [documentIsValid, setDocumentIsValid] = React.useState(true);
 
   const focusMovedOnSelectedDocumentChange = React.useRef<boolean>(false);
   React.useEffect(() => {
@@ -61,7 +59,7 @@ export const SchemaAsCodeEditor = ({
   const focusAndHomePosition = () => {
     if (editorRef.current) {
       editorRef.current.focus();
-      editorRef.current.setPosition({ column: 0, lineNumber: 1 });
+      editorRef.current.setPosition({ column: 1, lineNumber: 1 });
     }
   };
 
@@ -113,11 +111,13 @@ export const SchemaAsCodeEditor = ({
           });
         }
       }}
-      editorProps={{
-        onValidate: (markers) => {
-          setOkToSave(markers.every(({ severity }) => severity !== 8));
-        },
-      }}
+      // editorProps={{
+      //   onValidate: (markers) => {
+      //     setDocumentIsValid(
+      //       markers.every(({ severity }) => severity !== MarkerSeverity.Error)
+      //     );
+      //   },
+      // }}
       showEditor={!!currentCode}
       emptyState={
         <div className="simple-task-viewer-empty-state">

client/src/app/components/schema-defined-fields/SchemaAsFields.tsx

@@ -20,7 +20,6 @@ export interface SchemaAsFieldsProps {
   id: string;
   jsonDocument: object;
   jsonSchema: JsonSchemaObject;
-  onDocumentSaved?: (newJsonDocument: object) => void;
   onDocumentChanged?: (newJsonDocument: object) => void;
   isReadOnly?: boolean;
 }
@@ -222,7 +221,6 @@ export const SchemaAsFields: React.FC<SchemaAsFieldsProps> = ({
   jsonDocument,
   jsonSchema,
   onDocumentChanged,
-  // Note: onDocumentSaved doesn't make sense for this component
   isReadOnly = false,
 }) => {
   const { t } = useTranslation();

client/src/app/components/schema-defined-fields/SchemaDefinedFields.tsx

@@ -12,7 +12,6 @@ export interface ISchemaDefinedFieldProps {
   className?: string;
   jsonDocument: object;
   jsonSchema?: JsonSchemaObject;
-  onDocumentSaved?: (newJsonDocument: object) => void;
   onDocumentChanged?: (newJsonDocument: object) => void;
   isReadOnly?: boolean;
 }
@@ -22,20 +21,13 @@ export const SchemaDefinedField = ({
   className,
   jsonDocument,
   jsonSchema,
-  onDocumentSaved,
   onDocumentChanged,
   isReadOnly = false,
 }: ISchemaDefinedFieldProps) => {
   const [isJsonView, setIsJsonView] = React.useState<boolean>(
     !jsonSchema || isComplexSchema(jsonSchema)
   );
 
-  const onSavedHandler = !onDocumentSaved
-    ? undefined
-    : (newJsonDocument: object) => {
-        onDocumentSaved?.(newJsonDocument);
-      };
-
   const onChangeHandler = (newJsonDocument: object) => {
     onDocumentChanged?.(newJsonDocument);
   };
@@ -66,7 +58,6 @@ export const SchemaDefinedField = ({
             isReadOnly={isReadOnly}
             jsonDocument={jsonDocument}
             jsonSchema={jsonSchema}
-            onDocumentSaved={onSavedHandler}
             onDocumentChanged={onChangeHandler}
           />
         ) : (
@@ -75,7 +66,6 @@ export const SchemaDefinedField = ({
             isReadOnly={isReadOnly}
             jsonDocument={jsonDocument}
             jsonSchema={jsonSchema}
-            onDocumentSaved={onSavedHandler}
             onDocumentChanged={onChangeHandler}
           />
         )}