Skip to content

Stabilité: Ajout du composant c-title aux templates apply et approvals - #5921

Merged
hellodeloo merged 3 commits into
masterfrom
deloo/add-c-title-component-part03
Apr 14, 2025
Merged

Stabilité: Ajout du composant c-title aux templates apply et approvals#5921
hellodeloo merged 3 commits into
masterfrom
deloo/add-c-title-component-part03

Conversation

@hellodeloo

@hellodeloo hellodeloo commented Apr 9, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Après redécoupage de la fat PR slippers c-title composant. Voici la "petite" part03

🍰 Comment ?

Refactorisation de la zone de titre des templates apply et approvals afin d'utiliser le composant c-title.
Quelques corrections UI sur certains éléments de cette même zone.

@hellodeloo hellodeloo added the modifié Modifié dans le changelog. label Apr 9, 2025
@hellodeloo hellodeloo self-assigned this Apr 9, 2025
@hellodeloo hellodeloo changed the title feat: add c-title component to apply and approvals Ajout du composant c-title aux templates apply et approvals Apr 9, 2025
@hellodeloo
hellodeloo marked this pull request as draft April 9, 2025 13:50
@hellodeloo
hellodeloo force-pushed the deloo/add-c-title-component-part03 branch from 1947378 to 63fcd4b Compare April 9, 2025 14:01
@hellodeloo
hellodeloo marked this pull request as ready for review April 9, 2025 14:01
Comment thread itou/templates/apply/includes/job_applications_export_button.html
Comment thread itou/templates/apply/list_for_siae.html
Comment thread itou/templates/apply/list_of_available_exports.html
Comment thread itou/templates/apply/list_prescriptions.html
Comment thread itou/templates/apply/process_base.html
Comment thread itou/templates/apply/submit/application/base.html
Comment thread itou/templates/apply/submit/application/base.html
Comment thread itou/templates/approvals/details.html
Comment thread tests/www/apply/test_submit.py Outdated
Comment thread tests/www/apply/test_submit.py
@hellodeloo
hellodeloo force-pushed the deloo/add-c-title-component-part03 branch 2 times, most recently from fee1dc8 to a037e64 Compare April 14, 2025 11:55
@hellodeloo
hellodeloo requested review from EwenKorr and removed request for vincentporte April 14, 2025 12:26
@EwenKorr

Copy link
Copy Markdown
Contributor

Merci pour les explications et corrections ! 
À mon avis il ne reste qu'un w-100 à déplacer dans l'autre commit et c'est bon 👍

@hellodeloo
hellodeloo force-pushed the deloo/add-c-title-component-part03 branch from a037e64 to 0ad6a1d Compare April 14, 2025 15:03
@hellodeloo

hellodeloo commented Apr 14, 2025

Copy link
Copy Markdown
Contributor Author

Merci pour les explications et corrections ! À mon avis il ne reste qu'un w-100 à déplacer dans l'autre commit et c'est bon 👍

@EwenKorr Normalement, c'est bon

@EwenKorr

Copy link
Copy Markdown
Contributor

Super !
Et j'avais oublié de te dire pour les autres PR : il manque la catégorie dans le titre de la PR.
J'ai mis Stabilité dans le changelog mais il y a peut-être mieux comme choix.

@hellodeloo hellodeloo changed the title Ajout du composant c-title aux templates apply et approvals Stabilité: Ajout du composant c-title aux templates apply et approvals Apr 14, 2025
@hellodeloo
hellodeloo added this pull request to the merge queue Apr 14, 2025
Merged via the queue into master with commit 20d7271 Apr 14, 2025
@hellodeloo
hellodeloo deleted the deloo/add-c-title-component-part03 branch April 14, 2025 15:37
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