Skip to content
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

[Zone de vigilance] Ajout de la période sur l'infobulle des zones de vigilance #1912

Merged

Conversation

maximeperraultdev
Copy link
Collaborator

Related Pull Requests & Issues


  • Tests E2E (Cypress)

@maximeperraultdev maximeperraultdev force-pushed the maxime/feat/1875/add_periode_to_vigilance_area_overlay branch from d6a17ba to 87d9987 Compare December 4, 2024 10:30
@@ -5,7 +5,7 @@ import type { VigilanceArea } from '@features/VigilanceArea/types'
import type { AMPProperties } from 'domain/entities/AMPs'
import type { RegulatoryLayerCompactProperties } from 'domain/entities/regulatory'

export const getTitle = name => (name ? `${name?.replace(/[_]/g, ' ')}` : '')
export const getTitle = (name?: string | undefined | null) => (name ? `${name?.replace(/[_]/g, ' ')}` : '')
Copy link
Collaborator

Choose a reason for hiding this comment

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

pourquoi null, on est censés, avoir que des undefined. Si y'a des null qui trainent on peut les enlever je pense.

@maximeperraultdev maximeperraultdev force-pushed the maxime/feat/1875/add_periode_to_vigilance_area_overlay branch from ad3e44a to 9d18f47 Compare December 5, 2024 14:34
Copy link

sonarqubecloud bot commented Dec 5, 2024

@claire2212 claire2212 merged commit 74a6474 into main Dec 6, 2024
27 checks passed
@claire2212 claire2212 deleted the maxime/feat/1875/add_periode_to_vigilance_area_overlay branch December 6, 2024 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Zones de vigilance – Retouches sur carto
3 participants