[Tableau de bord] Ajout du formulaire de dessin de la zone et de la récupération des informations issue de la geometrie du dessin #1703
Conversation
maximeperraultdev
force-pushed
the
maxime/feat/1702/draw_form
branch
from
6227ad4
to
5c9f53a
Compare
maximeperraultdev
force-pushed
the
maxime/feat/1702/draw_form
branch
from
2ba8ae0
to
4d47572
Compare
maximeperraultdev
changed the title
maximeperraultdev
changed the title
maximeperraultdev
force-pushed
the
maxime/feat/1702/draw_form
branch
from
4d47572
to
01ef576
Compare
|
|
||
| describe('dashboard', () => { | ||
| it('should extract insee code, amps, regulatory and vigilance areas from the given geometry', () => { | ||
| Cypress.env('CYPRESS_FRONTEND_DASHBOARD_ENABLED', 'true') |
There was a problem hiding this comment.
je pense que ça fait doublon avec le beforeEach
There was a problem hiding this comment.
ça marche pas en plus ^^. c'est un reliquat, j'ai séparé en deux classes de test pour pas nous embêter à trier quand on enlevera le feature flag
| import type { Reporting } from './reporting' | ||
| import type { VigilanceArea } from '@features/VigilanceArea/types' | ||
|
|
||
| export namespace Dashboard { |
There was a problem hiding this comment.
l'archi front décidée est de créer un ficher types.ts à la racine de la feature
| return ( | ||
| <div className={className}> | ||
| <Header> | ||
| <Title as="h3">Définition d'une zone</Title> |
There was a problem hiding this comment.
pourquoi mettre as=h3 alors qu'il est défini comme un h2?
There was a problem hiding this comment.
je pensais que le style était différent mais j'ai du rêver.
| </Controls> | ||
|
|
||
| <div> | ||
| <CreateDashboardButton disabled={!isGeometryValid} onClick={handleValidate}> |
There was a problem hiding this comment.
le bouton doit être vert comme sur la maquette, et comme sur les autres modals de dessin de zone
| onClick={handleSelectInteraction(InteractionType.CIRCLE)} | ||
| /> | ||
| </li> | ||
| <ResetButton accent={Accent.SECONDARY} onClick={reinitialize}> |
There was a problem hiding this comment.
le bouton est pas aligné à droite :
There was a problem hiding this comment.
Si la maquette est pas à jour. C'est ma libre interpretation et je te previens, je suis pas un artiste 🗡️
| return null | ||
| } | ||
|
|
||
| function getOLTypeAndGeometryFunctionFromInteractionType(interactionType: InteractionType | null): { |
There was a problem hiding this comment.
peut-être pas à faire dans cette pr mais en fait on a trois fois la même fonction. Ca vaudrait le coup de la déplacer dans le dossier utils je pense
| @@ -0,0 +1,48 @@ | |||
| import { createSlice, type PayloadAction } from '@reduxjs/toolkit' | |||
There was a problem hiding this comment.
possible de déplacer le fichier à la racine de la feature pour être iso avec les autres features?
| dispatch( | ||
| addMainWindowBanner({ | ||
| children: `Une erreur est survenue lors de la génération du tableau de bord.`, | ||
| closingDelay: 10000, |
There was a problem hiding this comment.
je pense que tu peux enlever ce delay, 10 secondes ça ma paraît long. Par défaut c'est 3 secondes, je pense que ça suffit.
Et malgré une requête qui tombe en erreur je ne vois pas la bannière s'afficher quand je teste.
Pour résoudre le souci de la bannière qui n'appairait pas :
try {
const { data } = await dispatch(dashboardsAPI.endpoints.getExtratedArea.initiate(geometry))
if (!data) {
throw new Error()
}
dispatch(dashboardActions.setExtractedArea(data))
} catch (error) {
dispatch(
addMainWindowBanner({
children: `Une erreur est survenue lors de la génération du tableau de bord.`,
isClosable: true,
isFixed: true,
level: Level.ERROR,
withAutomaticClosing: true
})
)
}
| case 'MultiPoint': | ||
| return `MULTIPOINT (${geoJson.coordinates.map(coord => coord.join(' ')).join(', ')})` | ||
|
|
||
| case 'MultiLineString': |
There was a problem hiding this comment.
je suis pas sûre qu'on est ce type de géometrie de MonitorEnv, ni de LineString (enfin si pour les pointillés liés aux pop-up mais aucun besoin de les convertir en WTK)
There was a problem hiding this comment.
Mmmmh, je pense que c'est les géométries quand tu ajoutes plusieurs figures. Je peux vérifier ce qu'on envoie.
There was a problem hiding this comment.
Bon c'est tjs des mutlipolygones. Mais apres j'ai fait en sorte de respecter la spec geojson https://datatracker.ietf.org/doc/html/rfc7946
| .map(polygon => `((${polygon[0] && polygon[0].map(coord => coord.join(' ')).join(', ')}))`) | ||
| .join(', ')})` | ||
|
|
||
| case 'GeometryCollection': |
There was a problem hiding this comment.
idem on a ça comme Géometrie dans Env ?
|
@maximeperrault si je saisi une géometrie, le bouton "Créer le tableau" n'est plus disabled. Jusque là ok, mais si je clique sur "réinitialiser", le bouton est toujours actif et je peux cliquer dessus. La requête tombe en erreur et je ne vois pas de bannière.
|
maximeperraultdev
force-pushed
the
maxime/feat/1702/draw_form
branch
from
fb881b0
to
3fe4ae4
Compare
maximeperraultdev
force-pushed
the
maxime/feat/1702/draw_form
branch
from
43454e5
to
c321319
Compare
Related Pull Requests & Issues