[Carto] Ajout des zones de compétences des CROSS

[maximeperraultdev]

Related Pull Requests & Issues

• Resolve Positions CROSS - et compétences géographiques + tel/mail #805
• Resolve Zones administratives - Ajouter les zones de compétence des CROSS #864

---

• Tests E2E (Cypress)

[claire2212]

c'est normal qu'il ne soit pas entre "" comme pour "end"

[maximeperraultdev]

J'ai pas eu l'impression que ce soit problématique mais je peux le faire pour assurer le coup

[claire2212]

et pourquoi il y en a qui ont des quotes et d'autres non ?

[maximeperraultdev]

parce que c'est des mots réservés pour du SQL.

[maximeperraultdev]

https://www.postgresql.org/docs/current/sql-keywords-appendix.html

[claire2212]

il faut ajouter le style openlayer dans ce fichier : getAdministrativeLayersStyle. Et tu verras qu'on peut afficher du texte, il suffit d'ajouter la prop zoneFieldKey dans la définition du layer (fichier constants).

Par exemple pour les zones MARPOL :

case Layers.MARPOL.code:
      return feature =>
        new Style({
          stroke: new Stroke({
            color: darkPeriwinkle,
            width: 1
          }),
          text: new Text({
            fill: new Fill({ color: THEME.color.gunMetal }),
            font: '12px Marianne',
            stroke: new Stroke({ color: getColorWithAlpha(THEME.color.white, 0.9), width: 2 }),
            text: `${feature.get(Layers.MARPOL.subZoneFieldKey) ? feature.get(Layers.MARPOL.subZoneFieldKey) : ''}`
          })
        })

Je pense que pour les zones de compétences on peut utiliser ce qu'il y a dans la colonne name, donc à ajouter ici ;

  [MonitorEnvLayers.COMPETENCE_CROSS_AREA]: {
    code: 'competence_cross_areas',
    name: 'Zones de compétences des CROSS',
    type: LayerType.ADMINISTRATIVE,
    zoneFieldKey: 'name' 
  },

[VincentAntoine]

J'ai juste une petite interrogation sur le type de géométrie, c'est un LINESTRINGZ c'est-à-dire une géométrie 3D avec longitude, latitude et altitude. L'altitude est toujours 0 donc la dimension altitude ne sert à rien. Si ça marche comme ça ce n'est pas gênant, mais on pourrait la convertir en géométrie 2D, comme toutes les autres zones, par cohérence.

[VincentAntoine]

LGTM!
J'imagine que tu gardes deux fichiers de migration parce que le premier a déjà été déployé en inté ?

[maximeperraultdev]

@VincentAntoine Toutafé !