Skip to content

Stabilité: Ajout du composant c-title aux templates invitations, prescribers, siae et rdvi#5928

Merged
hellodeloo merged 2 commits intomasterfrom
deloo/add-c-title-component-part07
Apr 14, 2025
Merged

Stabilité: Ajout du composant c-title aux templates invitations, prescribers, siae et rdvi#5928
hellodeloo merged 2 commits intomasterfrom
deloo/add-c-title-component-part07

Conversation

@hellodeloo
Copy link
Copy Markdown
Contributor

@hellodeloo hellodeloo commented Apr 9, 2025

🤔 Pourquoi ?

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

🍰 Comment ?

Refactorisation de la zone de titre des templates invitations, prescribers, siae et rdvi 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 invitations, prescribers, siae and rdvi Ajout du composant c-title aux templates invitations, prescribers, siae et rdvi Apr 9, 2025
Copy link
Copy Markdown
Member

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

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

Merci pour la découpe, ça rend la revue plus gérable !

Comment thread itou/templates/rdv_insertion/discover.html Outdated
Comment thread itou/templates/siae_evaluations/campaign_calendar.html
Comment thread itou/templates/siae_evaluations/sanctions_helper.html
@hellodeloo hellodeloo force-pushed the deloo/add-c-title-component-part07 branch from 0b32786 to 9420780 Compare April 14, 2025 14:31
@hellodeloo hellodeloo changed the title Ajout du composant c-title aux templates invitations, prescribers, siae et rdvi Stabilité: Ajout du composant c-title aux templates invitations, prescribers, siae et rdvi Apr 14, 2025
@hellodeloo hellodeloo force-pushed the deloo/add-c-title-component-part07 branch from 9420780 to c7e76c9 Compare April 14, 2025 15:35
@hellodeloo hellodeloo force-pushed the deloo/add-c-title-component-part07 branch from c7e76c9 to 9df5ef5 Compare April 14, 2025 15:38
@francoisfreitag francoisfreitag force-pushed the deloo/add-c-title-component-part07 branch from 9df5ef5 to 06aa5af Compare April 14, 2025 15:50
Copy link
Copy Markdown
Member

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

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

Merci !
Je viens de push sur ta branche un revert des changements apportés (par inadvertance je suppose) aux fichiers requirements/*.txt.

@hellodeloo
Copy link
Copy Markdown
Contributor Author

Merci ! Je viens de push sur ta branche un revert des changements apportés (par inadvertance je suppose) aux fichiers requirements/*.txt.

Merci. Je ne comprenais pas d’où sortaient ces changements (j'imaginais d'un rebase master donc je ne voulais pas y toucher)

@hellodeloo hellodeloo added this pull request to the merge queue Apr 14, 2025
Merged via the queue into master with commit b4f2e01 Apr 14, 2025
11 checks passed
@hellodeloo hellodeloo deleted the deloo/add-c-title-component-part07 branch April 14, 2025 16:08
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