Skip to content

Candidature : permettre l'édition de la commune de naissance lors de l'acceptation d'une candidature, même si le candidat a des critères certifiés#6333

Merged
EwenKorr merged 1 commit into
masterfrom
ewen/gray_birth_city
Jun 16, 2025
Merged

Candidature : permettre l'édition de la commune de naissance lors de l'acceptation d'une candidature, même si le candidat a des critères certifiés#6333
EwenKorr merged 1 commit into
masterfrom
ewen/gray_birth_city

Conversation

@EwenKorr

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Les champs nécessaires à la certification des critères sont non modifiables, notamment lors de l'acceptation d'une candidature.
Or, parfois, le champ "Commune de naissance" est vide. Et donc le formulaire ne peut être validé.

🍰 Comment ?

On ne met ce champ en lecture seule que s'il n'est pas vide.

Since 1f63fcd we set the fields used to certify eligibility criteria
as readonly. However, sometimes job_seeker.jobseeker_profile.birth_place
is empty.
In that case we need to let the user fill a value, otherwise the form
remains invalid.
@EwenKorr EwenKorr added the modifié Modifié dans le changelog. label Jun 13, 2025
@EwenKorr EwenKorr self-assigned this Jun 13, 2025
@EwenKorr EwenKorr added the modifié Modifié dans le changelog. label Jun 13, 2025
@notion-workspace

Copy link
Copy Markdown

1 similar comment
@notion-workspace

Copy link
Copy Markdown

Comment thread itou/users/models.py
Comment on lines +1488 to +1489
if self.birth_place is None:
blocked_fields.discard("birth_place")

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Je n'ai pas systématisé à tous les champs pour ne pas avoir une surface de changement trop grande.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

J'aurais tendance au contraire à le mettre sur tous les champs 🤔
On a une idée de comment sont gérés les champs vides côté API ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

L'idée de ne faire que le champ de la commune de naissance c'est aussi d'être au courant par le support si d'autres champs ont ce problème.

@EwenKorr EwenKorr requested review from tonial and xavfernandez June 13, 2025 12:19
@EwenKorr EwenKorr added this pull request to the merge queue Jun 16, 2025
Merged via the queue into master with commit 5ad11f1 Jun 16, 2025
18 checks passed
@EwenKorr EwenKorr deleted the ewen/gray_birth_city branch June 16, 2025 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants