Skip to content

[Zone réglementaires] Modification de la gestion cartographique des zones réglementaires. - #2938

Open
maximeperraultdev wants to merge 8 commits into
mainfrom
maxime/feat/refacto_reg_area
Open

[Zone réglementaires] Modification de la gestion cartographique des zones réglementaires.#2938
maximeperraultdev wants to merge 8 commits into
mainfrom
maxime/feat/refacto_reg_area

Conversation

@maximeperraultdev

@maximeperraultdev maximeperraultdev commented May 20, 2026

Copy link
Copy Markdown
Collaborator

Modification de la gestion cartographique des zones réglementaires :

Pour l'affichage de toutes les zones on utilise des tuiles vectorielles (usage de ST_asMVT).
L'ancienne méthode de récupération des zones ne contient plus la géometrie mais uniquement l'étendue.
Pour l'affichage des zones spécifiques on garde les features openlayers.

Avantages:

Chargement rapides de toutes les zones reg sur la carto et sur la liste
Réduction de la charge réseau car les requêtes sont plus légères

Désavantages:

Les features sont immutables. Donc il faut conditionner le style et non modifier les propriétés des features

@tristanrobert

tristanrobert commented May 20, 2026

Copy link
Copy Markdown
Contributor

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@maximeperraultdev
maximeperraultdev force-pushed the maxime/feat/refacto_reg_area branch 7 times, most recently from 44f82d1 to 2bbbb52 Compare May 27, 2026 12:25
@maximeperraultdev maximeperraultdev changed the title feat: use ST_asMVT and add geom 3857 for regulatory areas [Zone réglementaires] Modification de la gestion cartographique des zones réglementaires. Jun 1, 2026
Comment thread frontend/src/features/Dashboard/hooks/useExportImages.tsx Outdated
skip: dashboard.regulatoryAreaIds.length === 0
}
)
const { data: regulatoryAreas } = useGetRegulatoryAreasByIdsQuery({

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pourquoi enlever skip: dashboard.regulatoryAreaIds.length === 0 ?

export type IsolatedLayerType = {
id: number
isFilled: boolean
isfilled: boolean

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pourquoi passer en minuscule?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Parce que les props renvoyées par le backend sont en minuscules et je ne sais pas comment les mettre en camelcase

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Finalement j'ai trouvé il fallait mettre des quotes...

@maximeperraultdev
maximeperraultdev force-pushed the maxime/feat/refacto_reg_area branch from 2bbbb52 to 0b23adf Compare June 1, 2026 13:11
@maximeperraultdev
maximeperraultdev force-pushed the maxime/feat/refacto_reg_area branch 2 times, most recently from c3b4f08 to e4037e0 Compare June 9, 2026 15:25
@sonarqubecloud

sonarqubecloud Bot commented Jun 9, 2026

Copy link
Copy Markdown

@maximeperraultdev
maximeperraultdev force-pushed the maxime/feat/refacto_reg_area branch from e4037e0 to a897ad1 Compare June 24, 2026 08:11
@maximeperraultdev
maximeperraultdev force-pushed the maxime/feat/refacto_reg_area branch 2 times, most recently from 3684339 to 20a8d57 Compare July 20, 2026 15:36
@maximeperraultdev
maximeperraultdev force-pushed the maxime/feat/refacto_reg_area branch from 20a8d57 to 3627d23 Compare July 21, 2026 06:47
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants