Skip to content

Candidature: ne plus demander les infos déjà disponibles pour les entreprises en IAE - #7096

Merged
xavfernandez merged 9 commits into
masterfrom
xf/only_ask_v2
Nov 18, 2025
Merged

Candidature: ne plus demander les infos déjà disponibles pour les entreprises en IAE#7096
xavfernandez merged 9 commits into
masterfrom
xf/only_ask_v2

Conversation

@xavfernandez

@xavfernandez xavfernandez commented Nov 10, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Car elles sont déjà disponibles.

🍰 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.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

🏝️ 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.

💻 Captures d'écran

@xavfernandez xavfernandez self-assigned this Nov 10, 2025
@xavfernandez xavfernandez added the modifié Modifié dans le changelog. label Nov 10, 2025
@xavfernandez
xavfernandez force-pushed the xf/only_ask_v2 branch 4 times, most recently from 484fbb7 to 150eeab Compare November 10, 2025 13:46
@xavfernandez xavfernandez added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Nov 10, 2025
@github-actions

github-actions Bot commented Nov 10, 2025

Copy link
Copy Markdown

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

EwenKorr
EwenKorr reviewed Nov 10, 2025
View reviewed changes

@EwenKorr EwenKorr left a comment

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.

Je n'ai lu que les 3 premiers commits, la suite devrait aller plus vite, j'ai l'impression qu'il y a un motif récurrent 😀

Comment thread tests/www/apply/test_process.py
@@ -2938,35 +2930,16 @@ def test_nir_readonly(self, client):
jobseeker_info_url = self.get_job_seeker_info_step_url(session_uuid)

@EwenKorr EwenKorr Nov 10, 2025

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.

Le NIR en lecture seule n'apparaîtra plus du coup ? Il y aura peut-être du nettoyage de tests à faire à la fin des travaux sur les formulaires.

@xavfernandez xavfernandez Nov 10, 2025

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.

Oui, je pense qu'il faudra refaire une passe sur tous nos tests car il y a surement des doublons 🙈

Comment thread tests/www/apply/test_process.py Outdated
Comment thread tests/www/apply/test_process.py
Comment thread tests/www/apply/test_submit.py
@xavfernandez
xavfernandez force-pushed the xf/only_ask_v2 branch 2 times, most recently from 1a0ac84 to 9486e7a Compare November 13, 2025 11:47
@xavfernandez
xavfernandez marked this pull request as ready for review November 13, 2025 11:47
EwenKorr
EwenKorr reviewed Nov 13, 2025
View reviewed changes

@EwenKorr EwenKorr left a comment

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.

Il me reste les deux derniers commits
C'est propre ! Je découvre assertFormError entre autres :)

Comment thread itou/www/apply/forms.py
Comment thread tests/www/apply/test_process.py Outdated
Comment thread itou/www/apply/forms.py Outdated
Comment thread tests/www/apply/test_process.py Outdated
EwenKorr
EwenKorr reviewed Nov 13, 2025
View reviewed changes

@EwenKorr EwenKorr left a comment

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.

Top ! J'ai fait quelques tests de mon côté, ça fonctionne bien.
Je tamponnerai quand j'aurai les réponses à mes questions :D

Comment thread tests/www/apply/test_process.py Outdated
Comment thread tests/www/apply/test_submit.py
@xavfernandez
xavfernandez force-pushed the xf/only_ask_v2 branch 6 times, most recently from e67629c to a6eb532 Compare November 14, 2025 15:59
@xavfernandez xavfernandez changed the title Candidature: ne plus demander les infos déjà disponibles Candidature: ne plus demander les infos déjà disponibles pour les entreprises en IAE Nov 14, 2025
@xavfernandez

xavfernandez commented Nov 14, 2025

Copy link
Copy Markdown
Contributor Author

@EwenKorr Normalement j'ai pris en compte tes remarques et répondu aux questions :)

EwenKorr
EwenKorr approved these changes Nov 17, 2025
View reviewed changes

@EwenKorr EwenKorr left a comment

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.

💯

Pour les tests, tu aurais une idée de réorganisation pour que ce soit un peu plus digeste et DRY ?

@xavfernandez
xavfernandez added this pull request to the merge queue Nov 18, 2025
@xavfernandez

xavfernandez commented Nov 18, 2025

Copy link
Copy Markdown
Contributor Author

Pour les tests, tu aurais une idée de réorganisation pour que ce soit un peu plus digeste et DRY ?

Pas forcément mais j'aimerais déjà sortir tous les tests en rapport avec le parcours d'embauche dans un test_hire.py

Merged via the queue into master with commit 1ee61b1 Nov 18, 2025
14 checks passed
@xavfernandez
xavfernandez deleted the xf/only_ask_v2 branch November 18, 2025 07:52
@xavfernandez xavfernandez mentioned this pull request Nov 21, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@EwenKorr EwenKorr EwenKorr approved these changes

Assignees

@xavfernandez xavfernandez

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié Modifié dans le changelog.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@xavfernandez @EwenKorr