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

feat(Activités des structures): Adaptation de l'affichage des secteurs d'activités des structures #1517

Merged
merged 6 commits into from
Nov 20, 2024

Conversation

SebastienReuiller
Copy link
Contributor

Quoi ?

Affichage des secteurs d'activité et types de prestation des activités sur les cartes des structures sur la recherche, sur la liste de favoris et sur la fiche structure

Pourquoi ?

Pour tenir compte du système des activités des structures.

Comment ?

Avec un template tag pour gérer l'affichage des secteurs d'activités. Dans la recherche, les secteurs sélectionnés doivent s'afficher en premier avec un style différent.

Et avec des accordéons sur la fiche structure.

Captures d'écran

Avant

screenshot-staging lemarche inclusion beta gouv fr-2024 11 13-18_25_23

Après

screenshot-marche localhost_8880-2024 11 13-18_24_32

Avant

screenshot-lemarche inclusion beta gouv fr-2024 11 13-18_28_51

Après

screenshot-marche localhost_8880-2024 11 13-18_29_29

Copy link
Contributor

Choose a reason for hiding this comment

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

Je peux toujours supprimer ce fichier dans ma PR sur la suppression des FlatPage ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui, complètement ! Je suis tombé dessus en cherchant "siae_sectors_display", mais je n'ai pas percuté que tu étais en train de le supprimer.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok nice, dans ce cas j'attends que tu merges cette PR avant de merge celle des flatpage :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok nice, dans ce cas j'attends que tu merges cette PR avant de merge celle des flatpage :)

@SebastienReuiller SebastienReuiller merged commit fa792d0 into master Nov 20, 2024
8 checks passed
@SebastienReuiller SebastienReuiller deleted the sreuiller/activities-switch branch November 20, 2024 21:57
chloend pushed a commit that referenced this pull request Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants