Skip to content

Interface: Corrections des écrans du parcours de modification d'une fiche structure - #6427

Merged
hellodeloo merged 2 commits into
masterfrom
deloo/update-edit-siae-templates-ui
Jul 7, 2025
Merged

Interface: Corrections des écrans du parcours de modification d'une fiche structure#6427
hellodeloo merged 2 commits into
masterfrom
deloo/update-edit-siae-templates-ui

Conversation

@hellodeloo

@hellodeloo hellodeloo commented Jun 27, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Pour appliquer la nouvelle UI

💻 Captures d'écran

Avant
👇
Screenshot 2025-06-30 at 11-40-03 Modifier les coordonnées de votre structure - Les emplois de l'inclusion

Apres
👇
Screenshot 2025-06-30 at 11-39-30 Modifier les coordonnées de votre structure - Les emplois de l'inclusion

@hellodeloo hellodeloo self-assigned this Jun 27, 2025
@hellodeloo hellodeloo added the modifié Modifié dans le changelog. label Jun 27, 2025
@hellodeloo
hellodeloo marked this pull request as draft June 27, 2025 14:51
@notion-workspace

notion-workspace Bot commented Jun 27, 2025

Copy link
Copy Markdown

💄 Amélioration UI parcours de modification fiche structure

@hellodeloo hellodeloo changed the title fix: update edit siae templates ui Interface: Amélioration parcours de modification d'une fiche structure Jun 27, 2025
@hellodeloo hellodeloo changed the title Interface: Amélioration parcours de modification d'une fiche structure Interface: Corrections des écrans du parcours de modification d'une fiche structure Jun 27, 2025
@hellodeloo
hellodeloo force-pushed the deloo/update-edit-siae-templates-ui branch from 3e27b28 to 26adf29 Compare June 30, 2025 09:38
@hellodeloo
hellodeloo marked this pull request as ready for review June 30, 2025 09:38
@hellodeloo hellodeloo added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Jun 30, 2025
@github-actions

github-actions Bot commented Jun 30, 2025

Copy link
Copy Markdown

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

EwenKorr
EwenKorr reviewed Jul 1, 2025
View reviewed changes

@EwenKorr EwenKorr left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ça me paraît ok !
Juste, je pense que ce serait pas mal de mutualiser les parties communes du gabarit (le titre, la boîte d'info de l'entreprise à droite etc).
On pourrait faire un nouveau gabarit, et à la place du {% extends "layout/base.html" %} on ferait un {% extends "companies/edit_base.html" %}, ou bien sinon des{% include %} pour le titre et la boîte d'info.

@hellodeloo

hellodeloo commented Jul 1, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Ça me paraît ok ! Juste, je pense que ce serait pas mal de mutualiser les parties communes du gabarit (le titre, la boîte d'info de l'entreprise à droite etc). On pourrait faire un nouveau gabarit, et à la place du {% extends "layout/base.html" %} on ferait un {% extends "companies/edit_base.html" %}, ou bien sinon des{% include %} pour le titre et la boîte d'info.

Effectivement. @EwenKorr Je te laisse rechecker

@hellodeloo
hellodeloo force-pushed the deloo/update-edit-siae-templates-ui branch from 26adf29 to 85732db Compare July 1, 2025 09:28
@hellodeloo
hellodeloo requested a review from EwenKorr July 1, 2025 09:50
EwenKorr
EwenKorr reviewed Jul 3, 2025
View reviewed changes
Comment thread itou/templates/companies/edit_siae_description.html Outdated
Comment thread itou/templates/companies/edit_siae_description.html Outdated
@hellodeloo
hellodeloo force-pushed the deloo/update-edit-siae-templates-ui branch from 85732db to 72b2710 Compare July 3, 2025 07:47
@hellodeloo
hellodeloo requested a review from EwenKorr July 3, 2025 08:04
francoisfreitag
francoisfreitag reviewed Jul 3, 2025
View reviewed changes

@francoisfreitag francoisfreitag left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Je suis assez de l’avis d’Ewen, on peut s’éviter pas mal de répétition sur le découpage. Je pousse un commit pour proposer une petite réorganisation des templates.

Comment thread tests/www/companies_views/test_edit_company_views.py Outdated
@francoisfreitag

francoisfreitag commented Jul 3, 2025

Copy link
Copy Markdown
Member

J’ai soumis ma revue par accident, je voulais ajouter que si jamais on doit ajouter une box à un des templates enfants, il suffira de créer un block sidebar (à mieux nommer 😉) qu’on pourra facilement étendre via un

{% block sidebar %}
{{ block.super }}
<div>Contenu additionnel
{% endblock sidebar %}

@francoisfreitag

francoisfreitag commented Jul 3, 2025

Copy link
Copy Markdown
Member

Qu’en pensez-vous ?

@hellodeloo
hellodeloo force-pushed the deloo/update-edit-siae-templates-ui branch 2 times, most recently from b3423ec to 9d3b117 Compare July 4, 2025 08:44
@hellodeloo

hellodeloo commented Jul 4, 2025

Copy link
Copy Markdown
Contributor Author

Qu’en pensez-vous ?

Moi ça me va. J'ai fixup les 2 commits et j'en ai profité pour ajouter un petit correctif au template pour les organisations

francoisfreitag
francoisfreitag reviewed Jul 4, 2025
View reviewed changes
Comment thread tests/www/companies_views/test_edit_company_views.py
response = client.post(step_2_url, data=post_data, follow=True)
attrs = 'target="_blank" rel="noopener" aria-label="Ouverture dans un nouvel onglet"'
assertContains(response, "Aperçu de la fiche")
assertContains(response, '<legend class="h3">Aperçu</legend>')

@francoisfreitag francoisfreitag Jul 4, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Je n’avais mis la suggestion qu’à un endroit, mais toutes les lignes changées de ce fichier pourraient inclure un peu plus de contexte de la même manière.

@EwenKorr

EwenKorr commented Jul 4, 2025

Copy link
Copy Markdown
Contributor

La nouvelle structure de edit_siae_base.html me va bien !
Il n'y a plus qu'à être un peu plus restrictif dans les assertContains et c'est bon pour moi. :)

@francoisfreitag
francoisfreitag force-pushed the deloo/update-edit-siae-templates-ui branch from 9d3b117 to 1055399 Compare July 7, 2025 07:26
francoisfreitag
francoisfreitag approved these changes Jul 7, 2025
View reviewed changes

@francoisfreitag francoisfreitag left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

J’ai ajouté le contexte aux tests où il manquait.

@hellodeloo
hellodeloo added this pull request to the merge queue Jul 7, 2025
Merged via the queue into master with commit dc6ea9e Jul 7, 2025
@hellodeloo
hellodeloo deleted the deloo/update-edit-siae-templates-ui branch July 7, 2025 09:13
@notion-workspace

notion-workspace Bot commented Jul 7, 2025

Copy link
Copy Markdown

💄 UX/UI: Amélioration des écrans de création de “Fiche de poste”

2 similar comments
@notion-workspace

notion-workspace Bot commented Jul 7, 2025

Copy link
Copy Markdown

💄 UX/UI: Amélioration des écrans de création de “Fiche de poste”

@notion-workspace

notion-workspace Bot commented Jul 7, 2025

Copy link
Copy Markdown

💄 UX/UI: Amélioration des écrans de création de “Fiche de poste”

@hellodeloo hellodeloo mentioned this pull request Jul 7, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@francoisfreitag francoisfreitag francoisfreitag approved these changes
@EwenKorr EwenKorr Awaiting requested review from EwenKorr

Assignees

@hellodeloo hellodeloo

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié Modifié dans le changelog.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hellodeloo @francoisfreitag @EwenKorr