Skip to content

[BUGFIX] Corriger le code postal de Paris 11 dans la table certification-cpf-cities (PIX-17457). #12030

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

Closed
wants to merge 1 commit into from

Conversation

Jeyffrey
Copy link
Contributor

@Jeyffrey Jeyffrey commented Apr 11, 2025

🌸 Problème

Lors de l’inscription d’un candidat à une certification dans Pix Certif, le champ lieu de naissance doit être renseigné, soit par la combinaison code postal + nom de la ville de naissance ou bien par le code INSEE de la ville de naissance du candidat.

Pour les candidats nés dans le 11e arrondissement de Paris, si on rentre “75011+Paris”, on obtient un code erreur alors que si on entre “75011+Paris 11” la ville de naissance est acceptée.

En cause, une ligne en BDD en erreur :
image

🌳 Proposition

Il se trouve que le code postal a un "0" en trop ici, le supprimer dans la table ceritification-cpf-cities via un script.

🤧 Pour tester

  • En local, créer une ligne dans la table ceritification-cpf-cities qui a 750011 comme postalCode.
  • Lancer le script
  • Vérifier que ce code postal est devenu 75011.

@Jeyffrey Jeyffrey self-assigned this Apr 11, 2025
@Jeyffrey Jeyffrey requested a review from a team April 11, 2025 09:24
@Jeyffrey Jeyffrey closed this Apr 11, 2025
@Jeyffrey Jeyffrey reopened this Apr 11, 2025
@1024pix 1024pix deleted a comment from pix-bot-github Apr 11, 2025
@alexandrecoin alexandrecoin changed the title [BUGFIX] Corriger le code postal de Paris 11 dans la table ceritification-cpf-cities (PIX-17457). [BUGFIX] Corriger le code postal de Paris 11 dans la table certification-cpf-cities (PIX-17457). Apr 11, 2025
@alexandrecoin
Copy link
Contributor

Requête effectuée par les captains.

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

Successfully merging this pull request may close these issues.

3 participants