-
Notifications
You must be signed in to change notification settings - Fork 1
[ZonesReg] Ajout de nouveaux champs sur les zones réglementaires #2679
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
7f3480a to
f746bd0
Compare
|
@claire2212 Dès que je clique sur une zone ça me dit qu'il y a une erreur : Aussi, j'ai l'impression que ce ne sont pas les bonnes couleurs de contour qui sont utilisées, tout à l'air d'utiliser la couleur Bright Green qui est lorsque l'on sélectionne une zone (Contour zone sélectionnée), les couleurs de contour sont les premières dans chaque colonne. Il a l'air d'y avoir des différences entre ce qui est affiché et la couleur dans le carré. Ici je survole la zone bleue foncée J'ai l'impression que les couleurs PIRC ne s'affichent pas de la bonne couleur, qu'elles s'affichent en PSCEM (tons verts/turquoises). Ou que c'est les couleurs pour PIRC+PSCEM. Ici pareil, survol de la grande zone turquoise mais le carré est bleu. Il y a aussi certains carrés qui apparaissent gris. En cliquant dessus l'app plante donc je n'arrive pas à voir si c'est justement le cas PIRC + PSCEM. |
331db3f to
73e6c63
Compare
|
maximeperraultdev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Une petite question non bloquante. 👍
| case Layers.REGULATORY_ENV.code: | ||
| if (plan === 'PIRC') { | ||
| colors = [regulatoryFishColors] | ||
| } else if (plan === 'PSCEM') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
question: est-ce qu'il n'y a que 2 valeurs possibles pour le plan ? Si oui est-ce que tu penses que çaa vaille le coup de sécuriser le type avec une enum.










Ajout des champs:
plan-> plan de contrôle (PIRC, PSCEM)resume-> remplace l'actuel entity_name (qui sera supprimé plus tard)poly_name-> titre concis de la zoneDans les différentes vue liste des zones reg, on affiche maintenant le
poly_namesi présent, sinon leresumeA modifier dans :
Related Pull Requests & Issues
resumeetpoly_namepour remplacer le champentity_name#2678