Skip to content

Candidature: Uniformisation du nom d'un paramètre GET#5971

Merged
tonial merged 1 commit into
masterfrom
alaurent/clean_kwargs
Apr 17, 2025
Merged

Candidature: Uniformisation du nom d'un paramètre GET#5971
tonial merged 1 commit into
masterfrom
alaurent/clean_kwargs

Conversation

@tonial

@tonial tonial commented Apr 16, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Ce changement va permettre d'utiliser le même nom de paramètre dans les différentes vues utilisées dans le parcours de candidature, et donc de simplifier la récupération de l'information.

Comme il n'y a pas de changement des schémas d'url mais juste d'un nom de paramètre GET, les or ... suffisent à garantir la rétrocompatibilité le temps que tout le monde soit déconnecté

🍰 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 added the modifié Modifié dans le changelog. label Apr 16, 2025
@tonial tonial self-assigned this Apr 16, 2025
Comment thread itou/www/search/views.py Outdated
Using job_seeker_public_id will allow us to plug make
RequireApplySessionMixin redirect to start view for session
initialization and then redirect to the same place as before
@tonial tonial force-pushed the alaurent/clean_kwargs branch from 5633af9 to 99f225d Compare April 17, 2025 08:29
@tonial tonial enabled auto-merge April 17, 2025 08:29
@tonial tonial added this pull request to the merge queue Apr 17, 2025
Merged via the queue into master with commit d6da17c Apr 17, 2025
@tonial tonial deleted the alaurent/clean_kwargs branch April 17, 2025 08:47
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