PR review

Georiviere · Apr 29, 2024 · 64ddb33 · 64ddb33
1 parent fe4a4fe
commit 64ddb33
Show file tree

Hide file tree

Showing 5 changed files with 21 additions and 15 deletions.
diff --git a/src/api/customObservations.ts b/src/api/customObservations.ts
@@ -1,5 +1,7 @@
 import { JSONSchema } from 'json-schema-yup-transformer/dist/schema';
 
+import { Attachement } from './settings';
+
 export type Observation = {
   id: number;
   label: string;
@@ -19,10 +21,10 @@ export type ObservationDetails = {
   };
 };
 
-type ObservationList = {
+type ObservationListItem = {
   id: number;
   contributed_at: string;
-  attachments: any[];
+  attachments: Attachement[];
 };
 
 async function fetchObservations(): Promise<Observation[]> {
@@ -59,7 +61,7 @@ async function fetchObservation(id: string): Promise<Observation | null> {
 
 async function fetchObservationDetails(
   id: string,
-): Promise<ObservationList[] | null> {
+): Promise<ObservationListItem[] | null> {
   const res = await fetch(
     `${process.env.apiHost}/api/portal/fr/${process.env.portal}/custom-contribution-types/${id}/contributions`,
     {
@@ -78,7 +80,14 @@ async function fetchObservationDetails(
 export async function getObservationDetails(
   type: string,
   id: string,
-): Promise<any> {
+): Promise<{
+  values: { id: string; value: any; label?: string }[];
+  id: string;
+  contributedAt: string;
+  label?: string;
+  description?: string;
+  attachments?: Attachement[];
+} | null> {
   const schema = await fetchObservation(type);
   const detailsList = await fetchObservationDetails(type);
   const values = detailsList?.find(detail => detail.id === parseInt(id));

diff --git a/src/api/postObservation.ts b/src/api/postObservation.ts
@@ -1,9 +1,3 @@
-export type PostObservationProps = {
-  geom?: string;
-  properties: string;
-  files?: string;
-};
-
 async function postObservation(
   props: { [key: string]: string | Blob },
   id: string,
@@ -53,7 +47,7 @@ async function postObservation(
 }
 
 export async function handleSubmitCustomObservation(
-  body: PostObservationProps,
+  body: { [key: string]: string | Blob },
   id: string,
   formData: FormData,
 ) {

diff --git a/src/app/[locale]/map/observation/[path]/page.tsx b/src/app/[locale]/map/observation/[path]/page.tsx
@@ -9,7 +9,7 @@ import { JSONSchema } from 'json-schema-yup-transformer/dist/schema';
 import { getTranslations } from 'next-intl/server';
 
 import ButtonClose from '@/components/button-close';
-import NewObservationForm from '@/components/new-observation-form';
+import CustomObservationForm from '@/components/custom-observation-form';
 import ObservationForm from '@/components/observation-form';
 
 type Props = {
@@ -60,7 +60,7 @@ export default async function ObservationPage({ params: { path } }: Props) {
           handleSubmit={handleSubmitObservation}
         />
       ) : (
-        <NewObservationForm
+        <CustomObservationForm
           id={observation.id}
           schema={jsonSchema}
           stations={observation.stations}

diff --git a/src/components/new-observation-form.tsx → src/components/custom-observation-form.tsx b/src/components/new-observation-form.tsx → src/components/custom-observation-form.tsx
@@ -20,7 +20,7 @@ import { useTranslations } from 'next-intl';
 import { Icons, propsForSVGPresentation } from './icons';
 import { Button } from './ui/button';
 
-const NewObservationForm = ({
+const CustomObservationForm = ({
   schema,
   id,
   stations,
@@ -194,4 +194,4 @@ const NewObservationForm = ({
   );
 };
 
-export default NewObservationForm;
+export default CustomObservationForm;
diff --git a/src/components/map.tsx b/src/components/map.tsx
@@ -45,11 +45,14 @@ export default function SearchMap() {
       const obs = await getObservation(id);
       if (obs?.stations?.length === 0) setHasObservationMarker(true);
     };
+
     if (pathName.startsWith('/map/observation')) {
       const observationType = pathName.match(
         /\/map\/observation\/([^?/]+)/,
       )?.[1];
+
       if (!observationType) return;
+
       if (DEFAULT_OBSERVATION_TYPES.includes(observationType)) {
         setHasObservationMarker(true);
       } else {