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.

Sign up for GitHub

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

Jump to bottom

[GEN-1599] Ajout d'un message d'alerte pour inviter à remplir une enquête sur la page PASS IAE #3969

Merged
merged 1 commit into from
Apr 25, 2024
Merged

[GEN-1599] Ajout d'un message d'alerte pour inviter à remplir une enquête sur la page PASS IAE #3969

merged 1 commit into from
Apr 25, 2024

Conversation

leo-naeka
Copy link
Contributor

@leo-naeka leo-naeka commented Apr 24, 2024 •
edited
Loading

🤔 Pourquoi ?

https://www.notion.so/plateforme-inclusion/Test-Maze-PASS-IAE-23ee84aa81444b26bf6e5e7cf0d2ba44

🍰 Comment ?

Ajout de deux alertes

💻 Captures d'écran

Candidature

Capture d’écran 2024-04-24 à 08 02 39

Profil salarié

Capture d’écran 2024-04-24 à 08 02 00

🚨 À vérifier

  • Ajouter l'étiquette « no-changelog » ?
  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

@leo-naeka leo-naeka added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC no-changelog Ne doit pas figurer dans le journal des changements. labels Apr 24, 2024
@leo-naeka leo-naeka self-assigned this Apr 24, 2024
@github-actions GitHub Actions
Copy link

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

@leo-naeka leo-naeka force-pushed the leo/alert-page-pass-iae branch 2 times, most recently from 2bc6195 to c3268af Compare April 25, 2024 13:46
@leo-naeka leo-naeka marked this pull request as ready for review April 25, 2024 14:13
xavfernandez
xavfernandez approved these changes Apr 25, 2024
View reviewed changes
Copy link
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 m'attendais à le voir dans le block messages.
Mais bon, que ce soit en haut du block messages ou en bas du block content_title, cela finit au même endroit 🤷‍♂️

itou/templates/apply/process_details_company.html Outdated Show resolved Hide resolved
@leo-naeka
Copy link
Contributor Author

Je m'attendais à le voir dans le block messages. Mais bon, que ce soit en haut du block messages ou en bas du block content_title, cela finit au même endroit 🤷‍♂️

Non t'as raison, ça a plus sa place avec les messages.

@leo-naeka leo-naeka force-pushed the leo/alert-page-pass-iae branch from c3268af to 8ced6a2 Compare April 25, 2024 14:46
@leo-naeka leo-naeka force-pushed the leo/alert-page-pass-iae branch from 8ced6a2 to 6689be8 Compare April 25, 2024 14:50
@leo-naeka leo-naeka added this pull request to the merge queue Apr 25, 2024
Merged via the queue into master with commit 4b34608 Apr 25, 2024
11 checks passed
@leo-naeka leo-naeka deleted the leo/alert-page-pass-iae branch April 25, 2024 15:14
@leo-naeka leo-naeka added ajouté Ajouté dans le changelog. and removed no-changelog Ne doit pas figurer dans le journal des changements. labels Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xavfernandez xavfernandez xavfernandez approved these changes

@rsebille rsebille Awaiting requested review from rsebille

Assignees

@leo-naeka leo-naeka

Labels
1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC ajouté Ajouté dans le changelog.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@leo-naeka @xavfernandez