Skip to content

feat: déplacement bandeau déploiement#299

Merged
ggounot merged 9 commits intomainfrom
feat/bandeau-deploiement
Mar 31, 2025
Merged

feat: déplacement bandeau déploiement#299
ggounot merged 9 commits intomainfrom
feat/bandeau-deploiement

Conversation

@ggounot
Copy link
Copy Markdown
Contributor

@ggounot ggounot commented Mar 24, 2025

  • Suppression de la bannière sur les résultats nationaux ;
  • Mise à jour de la bannière de déploiement déjà existante ;
  • Changements des règles d'affichage :
    • Si statut de déploiement du département antérieur à IN_PROGRESS ;
    • Si moins de 10 résultats avec sélection de filtre ;
    • Affichage après le 5ème résultat si possible, ou après le dernier résultat sinon.
  • Suppression de la bannière de déploiement de la page d'accueil ;
  • Refactoring complet et redesign du composant Notice.

Avant :
image

Après :
image

https://www.notion.so/gip-inclusion/Bandeau-d-ploiement-1995f321b60480c384a4c1bd2a8968c5

@ggounot ggounot force-pushed the feat/bandeau-deploiement branch 3 times, most recently from 74d0284 to ffeb5e1 Compare March 25, 2025 15:26
@ggounot ggounot force-pushed the feat/bandeau-deploiement branch from d330554 to 16a73dc Compare March 25, 2025 16:08
@ggounot ggounot marked this pull request as ready for review March 25, 2025 16:15
@ggounot ggounot requested a review from jbuget March 25, 2025 16:16
@jbuget
Copy link
Copy Markdown
Contributor

jbuget commented Mar 28, 2025

Feedback ETQ utilisateur : Je trouve que la notif quand il n'y a pas beaucoup de résultats n'est pas très adaptée.

Dans mon exemple ci-dessous, tant que je ne rajoutais pas le filtre thématique "Jeune sans soutien familial", j'avais plus d'une dizaine de résulats. Ca fait bizarre, tout-à-coup, d'avoir un message me disant que DORA n'est pas déployé sur le territoire alors qu'il me sortait des réponses juste avant.

image

Comment thread front/src/routes/recherche/dora-deployment-notice.svelte Outdated
@ggounot
Copy link
Copy Markdown
Contributor Author

ggounot commented Mar 28, 2025

Feedback ETQ utilisateur : Je trouve que la notif quand il n'y a pas beaucoup de résultats n'est pas très adaptée.

Dans mon exemple ci-dessous, tant que je ne rajoutais pas le filtre thématique "Jeune sans soutien familial", j'avais plus d'une dizaine de résulats. Ca fait bizarre, tout-à-coup, d'avoir un message me disant que DORA n'est pas déployé sur le territoire alors qu'il me sortait des réponses juste avant.
image

Oui je suis d'accord. Il faudrait en discuter avec le produit.

@ggounot ggounot force-pushed the feat/bandeau-deploiement branch from 28e4ca2 to 42beec6 Compare March 28, 2025 11:49
@ggounot ggounot force-pushed the feat/bandeau-deploiement branch from 42beec6 to dacf97e Compare March 28, 2025 11:50
Copy link
Copy Markdown
Contributor

@jbuget jbuget left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@ggounot ggounot merged commit 62f7b8f into main Mar 31, 2025
7 checks passed
@ggounot ggounot deleted the feat/bandeau-deploiement branch March 31, 2025 08:31
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