Show SubmitterId on edit page but do not allow it to be edited.

loculus-project · Jul 26, 2024 · b4629a8 · b4629a8
1 parent 42245d4
commit b4629a8
Show file tree

Hide file tree

Showing 5 changed files with 44 additions and 22 deletions.
diff --git a/kubernetes/loculus/templates/_inputFieldsFromValues.tpl b/kubernetes/loculus/templates/_inputFieldsFromValues.tpl
@@ -2,7 +2,7 @@
 {{- $data := . }}
 {{- $metadata := $data.metadata }}
 {{- $extraFields := $data.extraInputFields }}
-{{- $TO_KEEP := list "name" "displayName" "definition" "guidance" "example" "required" }}
+{{- $TO_KEEP := list "name" "displayName" "definition" "guidance" "example" "required" "noEdit"}}
 
 
 {{- $fieldsDict := dict }}

diff --git a/kubernetes/loculus/values.yaml b/kubernetes/loculus/values.yaml
@@ -748,6 +748,7 @@ defaultOrganismConfig: &defaultOrganismConfig
         guidance: Used to match the sequence(s) to the metadata
         example: GJP123
         position: first
+        noEdit: true
   preprocessing:
     - &preprocessing
       version: 1

diff --git a/website/src/components/Edit/EditPage.tsx b/website/src/components/Edit/EditPage.tsx
@@ -29,6 +29,20 @@ type EditPageProps = {
 
 const logger = getClientLogger('EditPage');
 
+type SubmissionProps = {
+    submissionId: string;
+};
+
+const SubmissionIdRow: FC<SubmissionProps> = ({ submissionId }) => (
+    <Fragment>
+        <tr>
+            <td className='w-1/4'>Submission ID:</td>
+            <td className='pr-3 text-right '></td>
+            <td className='w-full'>{submissionId}</td>
+        </tr>
+    </Fragment>
+);
+
 const InnerEditPage: FC<EditPageProps> = ({
     organism,
     dataToEdit,
@@ -102,10 +116,11 @@ const InnerEditPage: FC<EditPageProps> = ({
                     onConfirmation={submitEditedDataForAccessionVersion}
                 />
             </dialog>
-
             <table className='customTable'>
                 <tbody className='w-full'>
                     <Subtitle title='Original Data' bold />
+                    <Subtitle title='Non-Editable Metadata' />
+                    <SubmissionIdRow submissionId={dataToEdit.submissionId} />
                     <EditableOriginalData
                         editedMetadata={editedMetadata.filter(({ key }) => key !== ACCESSION_FIELD)}
                         setEditedMetadata={setEditedMetadata}
@@ -238,26 +253,30 @@ const EditableOriginalData: FC<EditableOriginalDataProps> = ({ editedMetadata, s
                 };
             }
 
-            return (
-                <EditableDataRow
-                    label={inputField.displayName ?? sentenceCase(inputField.name)}
-                    key={'raw_metadata' + inputField.name}
-                    row={field}
-                    onChange={(editedRow: Row) =>
-                        setEditedMetadata((prevRows: Row[]) => {
-                            const relevantOldRow = prevRows.find((oldRow) => oldRow.key === editedRow.key);
-
-                            if (relevantOldRow !== undefined) {
-                                return prevRows.map((prevRow) =>
-                                    prevRow.key === editedRow.key ? { ...prevRow, value: editedRow.value } : prevRow,
-                                );
-                            } else {
-                                return [...prevRows, editedRow];
-                            }
-                        })
-                    }
-                />
-            );
+            if (!(inputField.noEdit !== undefined && inputField.noEdit === true)) {
+                return (
+                    <EditableDataRow
+                        label={inputField.displayName ?? sentenceCase(inputField.name)}
+                        key={'raw_metadata' + inputField.name}
+                        row={field}
+                        onChange={(editedRow: Row) =>
+                            setEditedMetadata((prevRows: Row[]) => {
+                                const relevantOldRow = prevRows.find((oldRow) => oldRow.key === editedRow.key);
+
+                                if (relevantOldRow !== undefined) {
+                                    return prevRows.map((prevRow) =>
+                                        prevRow.key === editedRow.key
+                                            ? { ...prevRow, value: editedRow.value }
+                                            : prevRow,
+                                    );
+                                } else {
+                                    return [...prevRows, editedRow];
+                                }
+                            })
+                        }
+                    />
+                );
+            }
         })}
     </>
 );

diff --git a/website/src/types/backend.ts b/website/src/types/backend.ts
@@ -174,6 +174,7 @@ export const sequenceEntryToEdit = accessionVersion.merge(
     z.object({
         status: statusThatAllowsEditing,
         groupId: z.number(),
+        submissionId: z.string(),
         errors: z.array(processingAnnotation).nullable(),
         warnings: z.array(processingAnnotation).nullable(),
         originalData: z.object({

diff --git a/website/src/types/config.ts b/website/src/types/config.ts
@@ -43,6 +43,7 @@ export const metadata = z.object({
 export const inputField = z.object({
     name: z.string(),
     displayName: z.string().optional(),
+    noEdit: z.boolean().optional(),
 });
 
 export type InputField = z.infer<typeof inputField>;