Skip to content

Statistiques: Ajout d'une contrainte en base de donnée sur le champ processed_at des candidatures #4246

Merged
tonial merged 1 commit intomasterfrom
alaurent/job_application_denormalized_post_mep
Jun 25, 2024
Merged

Statistiques: Ajout d'une contrainte en base de donnée sur le champ processed_at des candidatures #4246
tonial merged 1 commit intomasterfrom
alaurent/job_application_denormalized_post_mep

Conversation

@tonial
Copy link
Copy Markdown
Contributor

@tonial tonial commented Jun 17, 2024 •
edited
Loading

🤔 Pourquoi ?

S'assurer que les données restes cohérentes

NE LIRE QUE LE DERNIER COMMIT : job_applications: Add constraint

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

💻 Captures d'écran

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

@tonial tonial added the ajouté Ajouté dans le changelog. label Jun 17, 2024
@tonial tonial self-assigned this Jun 17, 2024
@francoisfreitag francoisfreitag changed the title Statistiques: Ajout d'une contrainte en abse de donnée sur le champ débormalisé des candidatures Statistiques: Ajout d'une contrainte en base de donnée sur le champ dénormalisé des candidatures Jun 17, 2024
@francoisfreitag francoisfreitag changed the title Statistiques: Ajout d'une contrainte en base de donnée sur le champ dénormalisé des candidatures Statistiques: Ajout d'une contrainte en base de donnée sur le champ processed_at des candidatures Jun 17, 2024
@tonial tonial force-pushed the alaurent/job_application_denormalized_post_mep branch 4 times, most recently from b468eac to f2ee8f7 Compare June 18, 2024 13:48
@tonial tonial force-pushed the alaurent/job_application_denormalized_post_mep branch 4 times, most recently from e395355 to 34a98d4 Compare June 24, 2024 13:48
francoisfreitag
francoisfreitag approved these changes Jun 24, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

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

Ça vaudrait le coup de demander une recette jetable pour vérifier les principaux parcours.

@tonial
Copy link
Copy Markdown
Contributor Author

tonial commented Jun 24, 2024

Je suis super serein sur la contrainte vu la batterie de tests en tout genres qu'on a, et avec la présence des before_transition :)

@francoisfreitag
Copy link
Copy Markdown
Member

francoisfreitag commented Jun 24, 2024

Je suis super serein sur la contrainte vu la batterie de tests en tout genres qu'on a, et avec la présence des before_transition :)

Ce message sera conservé, et éventuellement ressorti. 🙈 📦

@tonial tonial force-pushed the alaurent/job_application_denormalized_post_mep branch from 34a98d4 to 7b0ed69 Compare June 24, 2024 19:47
@tonial tonial force-pushed the alaurent/job_application_denormalized_post_mep branch from 7b0ed69 to c13ebf5 Compare June 25, 2024 11:00
@tonial tonial added this pull request to the merge queue Jun 25, 2024
Merged via the queue into master with commit 0648d7b Jun 25, 2024
@tonial tonial deleted the alaurent/job_application_denormalized_post_mep branch June 25, 2024 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xavfernandez xavfernandez xavfernandez approved these changes

@francoisfreitag francoisfreitag francoisfreitag approved these changes

Assignees

@tonial tonial

Labels

ajouté Ajouté dans le changelog.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tonial @francoisfreitag @xavfernandez