Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Candidature: déplacer le bouton “annuler l’embauche” [GEN-2099] #4942

Merged
merged 2 commits into from
Oct 18, 2024

Conversation

xavfernandez
Copy link
Contributor

🤔 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

@xavfernandez xavfernandez self-assigned this Oct 17, 2024
@xavfernandez xavfernandez changed the title Candidature: déplacer le bouton “annuler l’embauche” Candidature: déplacer le bouton “annuler l’embauche” [GEN-2099] Oct 17, 2024
Copy link

@xavfernandez xavfernandez added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Oct 17, 2024
Copy link

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

@xavfernandez xavfernandez force-pushed the xfernandez/cancel_hire_modal branch 2 times, most recently from 6a9f120 to fbda1c4 Compare October 17, 2024 12:42
@xavfernandez xavfernandez marked this pull request as ready for review October 17, 2024 12:42
@tonial
Copy link
Contributor

tonial commented Oct 17, 2024

Je me rends compte qu'on a une autre page de confirmation après a modale :
image

Le contenu dépends déjà de si l'entreprise est une SIAE ou non, peut être que ça vaudrait le coup de ne pas mettre de modale en fait ?

À voir avec l'autre antoine vraissemblablement

@xavfernandez xavfernandez force-pushed the xfernandez/cancel_hire_modal branch 2 times, most recently from 8f8302e to cedcb8c Compare October 17, 2024 14:25
Copy link
Contributor

@hellodeloo hellodeloo left a comment

Choose a reason for hiding this comment

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

@xavfernandez pour infos, j'ai pushé un petit commit d'ajustements ui

@xavfernandez xavfernandez force-pushed the xfernandez/cancel_hire_modal branch 4 times, most recently from f76c515 to 3ec2bb5 Compare October 17, 2024 21:27
@xavfernandez
Copy link
Contributor Author

@tonial En fait, la demande était de faire sauter la page de confirmation d'annulation au profit de la modale uniquement (c'est ce qu'il y a dans ce ticket: https://www.notion.so/plateforme-inclusion/Transformer-les-pages-de-confirmations-en-modales-105e8fa5c35b80928641d0b5a6606ab4 ) : c'est ce qu'il y a dans cette nouvelle version.

@tonial
Copy link
Contributor

tonial commented Oct 18, 2024

👍

@xavfernandez xavfernandez added this pull request to the merge queue Oct 18, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 18, 2024
@xavfernandez xavfernandez added this pull request to the merge queue Oct 18, 2024
Merged via the queue into master with commit b97340a Oct 18, 2024
11 checks passed
@xavfernandez xavfernandez deleted the xfernandez/cancel_hire_modal branch October 18, 2024 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants