Skip to content

Candidature: Correction du formulaire de recherche de poste [GEN-2588] - #6467

Merged
tonial merged 1 commit into
masterfrom
alaurent/fix_form
Jul 3, 2025
Merged

tonial merged 1 commit into
masterfrom
alaurent/fix_form

Conversation

@tonial

@tonial tonial commented Jul 3, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Revert partiel de 0100b5b
On pourrait probablement adapter form_valid() pour gérer l'absence de donnée, mais c'est plus simple comme ça (et contrairement aux filtres de candidature, on n'aura pas le problème de classe is-valid avec des appels htmx)

🍰 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

@tonial
tonial requested a review from EwenKorr July 3, 2025 09:10
@tonial tonial self-assigned this Jul 3, 2025
@tonial tonial added modifié Modifié dans le changelog. bug labels Jul 3, 2025
@notion-workspace

Copy link
Copy Markdown

@EwenKorr

EwenKorr commented Jul 3, 2025

Copy link
Copy Markdown
Contributor

Je crois qu'il faudrait aussi corriger la page /search/prescribers/results

@tonial

tonial commented Jul 3, 2025

Copy link
Copy Markdown
Contributor Author

J'ai regardé, mais on ne peut pas y arriver sans être passé par /search/prescribers/home et le formulaire impose la présence d'une valeur

@tonial

tonial commented Jul 3, 2025

Copy link
Copy Markdown
Contributor Author

Bon, dans le doute je l'y remets aussi

@tonial
tonial force-pushed the alaurent/fix_form branch from 82c4f19 to 6ea3c84 Compare July 3, 2025 09:26
@tonial
tonial force-pushed the alaurent/fix_form branch from 6ea3c84 to b870f3e Compare July 3, 2025 09:27
@EwenKorr

EwenKorr commented Jul 3, 2025

Copy link
Copy Markdown
Contributor

J'ai regardé, mais on ne peut pas y arriver sans être passé par /search/prescribers/home et le formulaire impose la présence d'une valeur

Le menu de gauche y pointe directement

@tonial
tonial added this pull request to the merge queue Jul 3, 2025
Merged via the queue into master with commit b94966f Jul 3, 2025
@tonial
tonial deleted the alaurent/fix_form branch July 3, 2025 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants