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

[Tech] Mise à jour des propriétés front envoyées au back - Missions, Signalements, Backoffice #1473

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

claire2212
Copy link
Collaborator

Related Pull Requests & Issues


  • Tests E2E (Cypress)

@claire2212 claire2212 added the bug Something isn't working label Jun 19, 2024
@claire2212 claire2212 self-assigned this Jun 19, 2024
@claire2212 claire2212 changed the title [Tech] Mise à jour des propriétés front envoyées au back - Misions et Signalements [Tech] Mise à jour des propriétés front envoyées au back - Missions et Signalements Jun 19, 2024
@claire2212 claire2212 changed the title [Tech] Mise à jour des propriétés front envoyées au back - Missions et Signalements [Tech] Mise à jour des propriétés front envoyées au back - Missions, Signalements, Backoffice Jun 19, 2024
@maximeperraultdev maximeperraultdev force-pushed the maxime/feat/1378/endpoint_edition_action branch 2 times, most recently from b4d9486 to f8dc70b Compare June 27, 2024 16:55
Base automatically changed from maxime/feat/1378/endpoint_edition_action to main July 1, 2024 13:31
@maximeperraultdev maximeperraultdev force-pushed the claire/fix-frontend-backend-properties branch from 2f35a33 to 3ac1a23 Compare July 1, 2024 15:04
@@ -22,7 +23,7 @@ class MapperConfiguration {
mapper.disable(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS)
mapper.propertyNamingStrategy = PropertyNamingStrategies.LOWER_CAMEL_CASE
// TODO(15/06/2024): Activate this property when all front end API object are iso backend
// mapper.enable(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES)
mapper.enable(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES)
Copy link
Collaborator

Choose a reason for hiding this comment

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

todo: Il faut laisser que les autres équipes vérifient avec cette conf avant de merger.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants