Skip to content

GPS: Amélioration de l'affichage des résultat de l'ajout de bénéficiaires#5411

Merged
tonial merged 1 commit into
masterfrom
alaurent/gps_autocomplete
Jan 21, 2025
Merged

GPS: Amélioration de l'affichage des résultat de l'ajout de bénéficiaires#5411
tonial merged 1 commit into
masterfrom
alaurent/gps_autocomplete

Conversation

@tonial
Copy link
Copy Markdown
Contributor

@tonial tonial commented Jan 16, 2025

🤔 Pourquoi ?

Indiquez le problème que nous sommes en train de résoudre et les objectifs métiers ou techniques qui sont visés par ces changements.

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

🏝️ 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 added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié Modifié dans le changelog. labels Jan 16, 2025
@tonial tonial self-assigned this Jan 16, 2025
@github-actions
Copy link
Copy Markdown

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

Comment thread itou/static/js/gps.js Outdated
Comment on lines +41 to +43
escapeMarkup: function(markup) {
return markup;
}
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.

Et si tu renvoies un objet jQuery comme c'est actuellement le cas, escapeMarkup ne fait rien a priori (et donc autant garder sa valeur safe par défaut.

Comment thread itou/static/js/gps.js Outdated
Comment thread itou/www/autocomplete/views.py Outdated
@tonial tonial force-pushed the alaurent/gps_autocomplete branch 2 times, most recently from 730d943 to e417cf9 Compare January 17, 2025 09:18
Comment thread itou/static/js/gps.js Outdated
@tonial tonial force-pushed the alaurent/gps_autocomplete branch from e417cf9 to d213757 Compare January 17, 2025 10:20
@tonial tonial requested a review from xavfernandez January 17, 2025 13:39
@tonial tonial marked this pull request as ready for review January 17, 2025 13:51
@tonial tonial force-pushed the alaurent/gps_autocomplete branch 2 times, most recently from 22ef404 to 6f837f8 Compare January 17, 2025 13:53
@tonial tonial removed the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Jan 17, 2025
@EwenKorr EwenKorr added bug and removed bug labels Jan 17, 2025
@tonial tonial force-pushed the alaurent/gps_autocomplete branch from 6f837f8 to 3b18431 Compare January 17, 2025 20:40
Copy link
Copy Markdown
Contributor

@xavfernandez xavfernandez left a comment

Choose a reason for hiding this comment

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

Je pense qu'il y aura un soucis si ce champs est utilisé dans un formulaire pouvant échouer car User n'a pas de méthode python autocomplete_display, et si on devait en rajouter une, il faudrait qu'elle copie grosso modo le contenu de fonction JS format_result 😬 .
Tout cela pour dire qu'à ce moment là, cela vaudra ptet le coup de tout rapatrier en python 👀

Comment thread itou/www/autocomplete/views.py Outdated
@tonial
Copy link
Copy Markdown
Contributor Author

tonial commented Jan 21, 2025

Je peux mettre tout en python, mais ça veut dire escaper en python le nom et le titre, puis désactiver le escapeMarkup dans select2.
Je pars là dessus ?

Add a '.' after M, not Mme.
use DD/MM/YYYY date format
@xavfernandez
Copy link
Copy Markdown
Contributor

Non, la version actuelle me semble plus propre mais il faut avoir en tête ses limitations :)

@tonial tonial force-pushed the alaurent/gps_autocomplete branch from 3b18431 to cbb417b Compare January 21, 2025 09:05
@tonial tonial enabled auto-merge January 21, 2025 09:05
@tonial tonial added this pull request to the merge queue Jan 21, 2025
Merged via the queue into master with commit 9b6062b Jan 21, 2025
@tonial tonial deleted the alaurent/gps_autocomplete branch January 21, 2025 09:25
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.

3 participants