-
Notifications
You must be signed in to change notification settings - Fork 0
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
[BO - Liste signalement] Pouvoir filtrer la liste des signalement à travers l'url #2686
Conversation
update dashboard link based on feature flipping #2120
a40b15c
to
17cd285
Compare
handleQueryParameter () { | ||
const url = new URL(window.location.toString()) | ||
const params = new URLSearchParams(url.search) | ||
const page = params.get('page') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
est-ce que tout cette initialisation pourrait être dans un fichier à part ?
Dans filters.ts
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oui on pourrait mais je ferai plutôt un fichier à part.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
un fichier en plus de filters ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C'est cohérent d'exposer une fonction supplémentaire dans une interface comme filters ? Bien que cela soit techniquement possible, je me demande si cela respecte les bonnes pratiques. C'est mon coté PHP qui parle
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je ne sais pas trop pour les bonnes pratiques :)
Quoiqu'il en soit, c'est un peu long je trouve pour ce genre de méthode.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai une question pour l'instant, je n'ai pas encore testé.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai noté quelque points à revoir
En SUPERADMIN
- Lorsque l'on colle un URL l'autocompletion du champ "commune ou code postal" ne fonctionne plus correctement
- Idem pour le champ EPCI, Etiquette, Partenaire
Test KO
- http://localhost:8080/bo/v2/signalements/?allocataire=caf&sortBy=reference&direction=DESC : quand je change la valeur via le champ ca génère un URL en erreur http://localhost:8080/bo/v2/signalements/?status=caf&allocataire=msa&sortBy=reference&direction=DESC
- http://localhost:8080/bo/v2/signalements/?situation=bail_en_cours&sortBy=reference&direction=DESC : URL en erreur
Mineur
- Lorsque l'on colle un URL contenant une recherche EPCI son badge ne contient plus son nom mais son code (comme c'est celui qui est utilisé dans l'URL)
Questions
- Ne faudrait il pas ouvrir la section des filtres supplémentaire quand on colle un URL en contenant ?
- Ne faudrait il pas fermer la section des filtres supplémentaire quand on clique sur le bouton de réinitialisation ?
|
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai toujours les 3 points ci dessous, mais vu que c'est uniquement en SUPERADMIN, c'est pas important, je valide.
- Lorsque l'on colle un URL l'autocompletion du champ "commune ou code postal" ne fonctionne plus correctement
- Idem pour le champ EPCI
- Etiquette, Partenaire (J'ai pas réussi à reproduire)
assets/vue/components/signalement-list/TheSignalementAppList.vue
Outdated
Show resolved
Hide resolved
assets/vue/components/signalement-list/TheSignalementAppList.vue
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK lecture et tests
Ticket
#2636
#2120
Description
Permettre la recherche depuis la barre d'adresse
Faire pointer le tableau de bord sur la nouvelle liste lorsque la variable d'environnement est activée
Changements apportés
Pré-requis
Supprimer tous les pools de cache
make clear-pool pool="--all"
make npm-build # ou make npm-watch
Les tests sont à faire avec tous les profils.
Tests
Filtre barre d'adresse
Filtre tableau de bord
Test de non régression
make clear-pool pool="--all"