Skip to content

PASS IAE: correction des boutons d'annulation/retour dans le processus de suppression d'une suspension#4835

Merged
xavfernandez merged 1 commit intomasterfrom
xfernandez/propagate_back_url
Sep 27, 2024
Merged

PASS IAE: correction des boutons d'annulation/retour dans le processus de suppression d'une suspension#4835
xavfernandez merged 1 commit intomasterfrom
xfernandez/propagate_back_url

Conversation

@xavfernandez
Copy link
Copy Markdown
Contributor

@xavfernandez xavfernandez commented Sep 27, 2024

🤔 Pourquoi ?

Il y avait un mélange des back_url/reset_url/secondary_url qui empêchait l'utilisateur de revenir de là où il venait.

Le mieux est encore de comparer la différence de comportement entre cette recette jetable et la démo.

🍰 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

@xavfernandez xavfernandez added the modifié Modifié dans le changelog. label Sep 27, 2024
@xavfernandez xavfernandez requested a review from tonial September 27, 2024 13:31
@xavfernandez xavfernandez self-assigned this Sep 27, 2024
@xavfernandez xavfernandez added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Sep 27, 2024
return HttpResponseRedirect(
reverse("employees:detail", kwargs={"public_id": suspension.approval.user.public_id})
)
return HttpResponseRedirect(back_url)

Check warning

Code scanning / CodeQL

URL redirection from remote source

Untrusted URL redirection depends on a [user-provided value](1).
@xavfernandez xavfernandez added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Sep 27, 2024
@xavfernandez xavfernandez removed the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Sep 27, 2024
@xavfernandez xavfernandez added this pull request to the merge queue Sep 27, 2024
Merged via the queue into master with commit d360470 Sep 27, 2024
@xavfernandez xavfernandez deleted the xfernandez/propagate_back_url branch September 27, 2024 14:39
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