Fix zone director form value

The form field path name needs to match the GQL input path name.
This is what the API uses when there are certain field errors.
This lets us transparently show API errors on certain fields.

Like for example in this case, we can show that the user
does not have the required director role.

Author: CarsonF

src/components/FieldZone/CreateFieldZone/CreateFieldZone.tsx

@@ -29,7 +29,7 @@ export const CreateFieldZone = (props: CreateFieldZoneProps) => {
         const input: CreateFieldZoneInput = {
           fieldZone: {
             name: values.fieldZone.name,
-            directorId: values.fieldZone.director!.id,
+            directorId: values.fieldZone.directorId!.id,
           },
         };
 

src/components/FieldZone/EditFieldZone/EditFieldZone.tsx

@@ -25,7 +25,7 @@ export const EditFieldZone = (
       fieldZone: {
         id: fieldZone.id,
         name: fieldZone.name.value,
-        director: fieldZone.director.value ?? undefined,
+        directorId: fieldZone.director.value ?? null,
       },
     }),
     [fieldZone]
@@ -41,7 +41,7 @@ export const EditFieldZone = (
           fieldZone: {
             id: values.id,
             name: values.name,
-            directorId: values.director?.id ?? null,
+            directorId: values.directorId?.id,
           },
         };
 

src/components/FieldZone/FieldZoneForm/FieldZoneForm.graphql

@@ -9,8 +9,7 @@ fragment FieldZoneForm on FieldZone {
     canEdit
     canRead
     value {
-      id
-      fullName
+      ...UserLookupItem
     }
   }
 }

src/components/FieldZone/FieldZoneForm/FieldZoneForm.tsx

@@ -14,7 +14,7 @@ export interface FieldZoneFormValues<Mutation extends FieldZoneMutation> {
   fieldZone: Merge<
     Mutation,
     {
-      director?: Pick<UserLookupItemFragment, 'id' | 'fullName'>;
+      directorId: UserLookupItemFragment | null;
     }
   >;
 }
@@ -45,9 +45,7 @@ export const FieldZoneForm = <Mutation extends FieldZoneMutation, R>({
           )}
         </SecuredField>
         <SecuredField obj={fieldZone} name="directorId">
-          {(props) => (
-            <UserField {...props} name="director" label="Director" required />
-          )}
+          {(props) => <UserField {...props} label="Director" required />}
         </SecuredField>
       </Stack>
     </FieldGroup>