Skip to content

Conversation

@louije
Copy link
Contributor

@louije louije commented Jul 3, 2024

🤔 Pourquoi ?

Les utilisateurs GPS demandent une fonction de recherche / filtrage de la liste des bénficiaires. C'est un certain boulot, donc en attendant, devant nos urgences en déploiement / com, on ajoute un bandeau avec des infos basiques.

🍰 Comment ?

Juste un bandeau en plus du bandeau déjà existant (😓).

🚨 À vérifier

  • Il faudra le supprimer lors de la publication de la version avec filtres.

🏝️ Comment tester

N/A.

💻 Captures d'écran

image

@louije louije requested a review from celine-m-s July 3, 2024 15:00
@louije louije added html modifié Modifié dans le changelog. and removed html labels Jul 3, 2024
@louije
Copy link
Contributor Author

louije commented Jul 3, 2024

Alors c'est ma première ici, je savais pas vraiment quels labels mettre (et je me suis pas encore organisé pour faire tourner make quality sur ma machine).

Copy link
Contributor

@celine-m-s celine-m-s left a comment

Choose a reason for hiding this comment

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

Bravo pour ta première PR sur les emplois !!! 💃
J'ai suggéré une amélioration facultative mais la PR me semble prête à l'emploi.

louije added 4 commits July 4, 2024 09:54
Add temporary info banner on the GPS user list, with instructions on using the browser “Find” command.
@celine-m-s celine-m-s force-pushed the louije/add-gps-find-user-banner branch from 5487420 to ca2568f Compare July 4, 2024 07:54
@celine-m-s celine-m-s enabled auto-merge July 4, 2024 07:54
@celine-m-s celine-m-s added this pull request to the merge queue Jul 4, 2024
Merged via the queue into master with commit e873077 Jul 4, 2024
@celine-m-s celine-m-s deleted the louije/add-gps-find-user-banner branch July 4, 2024 08:09
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.

3 participants