feat: signalement V1 #919

MaGOs92 · 2024-05-24T14:41:09Z

Modifications :

Ajout d'un proxy pour authentifier les résolutions de signalements tout en gardant le token côté serveur.
Mise à jour du client Open API de l'API de signalement (migration Postgrest)

fufeck · 2024-12-02T11:10:07Z

Il faudrait ajouter la procédure d'installation de mes-signalement et api-signalement.
Les .env.sample sont faux et les README en retard

Il faut mettre la var SYNCHRONIZE_DB a true dans le .env de api-signalement pour installé la DB, c'est pas évident
Ensuite il faut créer une source qu'on rajoute coté mes-signalement
Et créer un client pour coller le token coté mes-adresses

fufeck · 2024-12-02T11:21:51Z

Je ne peux pas créer de signalement en local

components/base-locale-card/index.tsx

fufeck · 2024-12-02T11:36:54Z

components/map/editable-marker.tsx

-            : `${marker.type}`}
-        </Text>
+  return markers
+    .filter((marker) => !marker.isMapMarker)


je n'arrive pas a comprend a quoi sert le isMapMarker et dans quelle cas il est false ?

J'ai ajouté ce flag pour pouvoir utiliser le même contexte "markers" déjà utilisé dans l'app. Il est à false pour les markers utilisés editable-markers et à true pour ceux des signalements.

components/signalement/signalement-diff/signalement-parcelle-diff.tsx

fufeck · 2024-12-02T14:18:56Z

components/signalement/signalement-diff/signalement-toponyme-diff-card.tsx

+import { Heading, Pane } from "evergreen-ui";
+import TextDiff from "./text-diff";
+
+interface SignalementToponymeDiffCardProps {
+  title: string;
+  nom: {
+    from?: string;
+    to: string;
+  };
+  backgroundColor?: string;
+}
+
+export function SignalementToponymeDiffCard({
+  title,
+  nom,
+  backgroundColor,
+}: SignalementToponymeDiffCardProps) {
+  return (
+    <Pane
+      background={backgroundColor || "white"}
+      padding={8}
+      borderRadius={8}
+      marginBottom={8}
+      width="100%"
+    >
+      <Heading is="h3" marginBottom={8}>
+        {title}
+      </Heading>
+      <Pane padding={8} borderRadius={8} className="glass-pane">
+        <TextDiff from={nom.from} to={nom.to} />
+      </Pane>
+    </Pane>
+  );
+}


Je ne vois pas la différence avec le component SignalementVoieDiffCardProps

Oula bien vu j'ai oublié de mettre les positions et les parcelles pour les toponymes. Je rajoute

components/signalement/signalement-viewer/numero/signalement-viewer-create-numero.tsx

components/signalement/signalement-form/numero/signalement-create-numero.tsx

components/signalement/signalement-form/signalement-form.tsx

fufeck · 2024-12-05T10:50:35Z

hooks/fuse.ts

+    if (!fuse) {
+      return;
    }
-  }, delay);
+
+    if (value) {
+      const results = fuse.search(value);
+      setFiltered(results.map((result) => result.item));
+    } else {
+      setFiltered(source);
+    }
+  }, [fuse, value, source]);
+
+  const debouncedCallback = useDebouncedCallback(
+    (value) => setValue(value),
+    delay
+  );


Pour moi ca complexifie le useFuse, je trouve ca beaucoup moins clair si cela fait la même chose

Oui je suis d'accord, j'ai été obligé de faire comme ça pour ajouter la fonctionnalité de filtre par statut / nom sur la page des signalements.
Sans cette modif, ça faisait une boucle infinie avec un useEffect

fufeck

Je trouve que la couleur des diff est très peu (surtout les nom)

Et parfois je ne comprends juste pas la diff

Il n'y a pas de Synchro en cours lorsque je met un signalement a jour, est ce normale ?

Lorsque l’on est dans un signalement archivé et que l’on click sur annuler cela ramène a en cours (au lieu de Archivés)

Problème avec les modification de toponyme

Je le trouve bien dans mes-adressses

Et quand je clique dessus j'arrive sur cette page

MaGOs92 · 2024-12-06T15:52:05Z

Il n'y a pas de Synchro en cours lorsque je met un signalement a jour, est ce normale ?
=> Yes bien vu, j'avais oublié le "refreshBALSync"
Lorsque l’on est dans un signalement archivé et que l’on click sur annuler cela ramène a en cours (au lieu de Archivés)
=> Ok j'ai ajouté des queryParams pour ne pas perdre la dernière tab visitée (comme dans ta PR)
Problème avec les modification de toponyme
=> C'est bizarre, normalement qu'en tu arrives sur cette page c'est qu'il n'a pas réussi à trouver le lieu dans la BAL. Est-ce que le toponyme n'avait pas été modifié par un autre signalement avant?

fufeck · 2024-12-18T09:42:54Z

components/breadcrumbs.tsx

@@ -24,7 +23,7 @@ function Breadcrumbs({
  ...props
 }: BreadcrumbsProps) {
  const router = useRouter();
-  const { signalements } = useContext(SignalementContext);
+  const { breadcrumbs } = useContext(LayoutContext);


Attention avec le merge de la master actuelle, j'ai touché au breadcrumbs pour la navigation

fufeck · 2024-12-18T09:45:47Z

components/map/layers/cadastre.ts

+        "rgba(244, 228, 219, 1)",
+        ["==", ["feature-state", "diff"], SignalementDiff.NEW],
+        "rgba(218, 244, 246, 1)",
+        "rgba(200, 200, 200, 1)",


Pourquoi ne pas utiliser des variable pour les couleur?
Je pinaille beaucoup ^^

fufeck · 2024-12-18T09:47:48Z

contexts/map.tsx

@@ -47,6 +50,7 @@ export const SOURCE_TILE_ID = "tiles";

 export function MapContextProvider(props: ChildrenProps) {
  const [map, setMap] = useState<MaplibreMap | null>(null);
+  const [showToponymes, setShowToponymes] = useState<boolean>(true);


Je ne comprends pas a quoi sert ce nouveau state, je le vois set null part ?

fufeck · 2024-12-18T09:51:43Z

pages/bal/[balId]/toponymes/[idToponyme].tsx

-  const [filtered, setFilter] = useFuse(numeros, 200, {
-    keys: ["numero"],
-  });
+  const [filtered, setFilter] = useFuse(numeros, 200, fuseOptions);


Attention au merge avec la master (PR sur la navigation)

fufeck · 2024-12-18T09:58:40Z

contexts/parcelles.tsx

+  const { baseLocale, parcelles: selectedParcelles } =
+    useContext(BalDataContext);
+  const [showSelectedParcelles, setShowSelectedParcelles] =
+    useState<boolean>(true);
+  const [isDiffMode, setIsDiffMode] = useState<boolean>(false);
  const [hoveredParcelle, setHoveredParcelle] = useState<{
    id: string;
    featureId?: string;
  } | null>(null);
  const [isParcelleSelectionEnabled, setIsParcelleSelectionEnabled] =
    useState<boolean>(false);
-  const [selectedParcelles, setSelectedParcelles] = useState<string[]>([]);
+  const [highlightedParcelles, setHighlightedParcelles] = useState<string[]>(
+    []
+  );


J'ai du mal a comprendre pourquoi renomer selectedParcelles en highlightedParcelles et rajouter showSelectedParcelles ?
De plus tu renome parcelles en selectedParcelles mais c'est bien les parcelles selectionnée de la BAL ?

fufeck · 2024-12-18T10:03:39Z

contexts/token.tsx

@@ -33,7 +33,7 @@ export function TokenContextProvider({
 }: TokenContextProviderProps) {
  const { getBalToken, addBalAccess } = useContext(LocalStorageContext);

-  const [tokenIsChecking, setTokenIsChecking] = useState<boolean>(false);
+  const [tokenIsChecking, setTokenIsChecking] = useState<boolean>(true);


C'est faux non, le token n'a encore été checker a l'init, pourquoi on a besoin de l'init a true ?

MaGOs92 force-pushed the gfay_feat_signalement-client branch 2 times, most recently from 7785fd9 to c0e25fb Compare June 18, 2024 13:30

MaGOs92 force-pushed the gfay_feat_signalement-client branch from a83eee4 to 2910ec7 Compare July 11, 2024 15:03

MaGOs92 marked this pull request as ready for review July 11, 2024 15:03

MaGOs92 requested a review from fufeck July 17, 2024 07:41

MaGOs92 force-pushed the gfay_feat_signalement-client branch from d9b3789 to 8d0ba3d Compare July 17, 2024 07:41

MaGOs92 force-pushed the gfay_feat_signalement-client branch from 6d2234c to 1a55372 Compare September 20, 2024 09:43

MaGOs92 force-pushed the gfay_feat_signalement-client branch 2 times, most recently from 72507e0 to 6e9e975 Compare October 25, 2024 09:34

MaGOs92 force-pushed the gfay_feat_signalement-client branch from 1d5f48b to 2391693 Compare November 14, 2024 15:20

MaGOs92 force-pushed the gfay_feat_signalement-client branch from 8b7c0d8 to 6883370 Compare November 28, 2024 16:59

MaGOs92 changed the title ~~feat: add signalement client authentification~~ feat: signalement V1 Nov 29, 2024

fufeck reviewed Dec 5, 2024

View reviewed changes

MaGOs92 force-pushed the gfay_feat_signalement-client branch from 80531e1 to 4346557 Compare December 6, 2024 15:54

MaGOs92 force-pushed the gfay_feat_signalement-client branch 2 times, most recently from 91973fd to 7baa08d Compare December 17, 2024 17:16

MaGOs92 requested a review from fufeck December 18, 2024 09:07

fufeck approved these changes Dec 18, 2024

View reviewed changes

MaGOs92 added 8 commits December 18, 2024 17:20

feat: signalement v1

a4eb222

fix: no white list if the env var is undefined

340ae21

fix: request signalements context ignored instead of expired

064cdf5

feat: retour atelier signalement Thumeries

0817327

fix product tour

75fc26b

fix: product tour

0374951

fix: wrong fly to for creations

1db4bf9

fix: infinite loop on create adresse

b717a93

fix: add hook for map loaded

995d9a0

MaGOs92 force-pushed the gfay_feat_signalement-client branch from 55cb9a7 to 995d9a0 Compare December 20, 2024 15:37

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: signalement V1 #919

feat: signalement V1 #919

MaGOs92 commented May 24, 2024 •

edited

Loading

fufeck commented Dec 2, 2024 •

edited

Loading

fufeck commented Dec 2, 2024

fufeck Dec 2, 2024

MaGOs92 Dec 5, 2024

fufeck Dec 2, 2024

MaGOs92 Dec 5, 2024

fufeck Dec 5, 2024

MaGOs92 Dec 5, 2024

fufeck left a comment

MaGOs92 commented Dec 6, 2024

fufeck Dec 18, 2024

fufeck Dec 18, 2024

fufeck Dec 18, 2024

fufeck Dec 18, 2024

fufeck Dec 18, 2024

fufeck Dec 18, 2024

feat: signalement V1 #919

Are you sure you want to change the base?

feat: signalement V1 #919

Conversation

MaGOs92 commented May 24, 2024 • edited Loading

fufeck commented Dec 2, 2024 • edited Loading

fufeck commented Dec 2, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

fufeck left a comment

Choose a reason for hiding this comment

MaGOs92 commented Dec 6, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

MaGOs92 commented May 24, 2024 •

edited

Loading

fufeck commented Dec 2, 2024 •

edited

Loading