-
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 - Filtre signalement] Correction tri #2617
Conversation
@@ -172,9 +172,9 @@ export default defineComponent({ | |||
} |
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.
eslint fix
@@ -95,7 +95,7 @@ export const requests = { | |||
// TODO : il y a sûrement plus élégant à faire pour construire l'url (cf controlleur et twig) |
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.
eslint fix
foreach ($signalements as $signalement) { | ||
$createdAtUpdated = $signalement->getCreatedAt()->modify('+'.$second.' second'); |
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.
Pour test tri date de dernier suivi avec fixtures (pour avoir quelques dates différentes :-) )
private readonly string $sortBy = 'reference', | ||
private readonly string $orderBy = 'DESC', | ||
) { | ||
} | ||
|
||
public function getTerritoires(): ?array | ||
public function getTerritoire(): ?string |
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.
S'en tenir à la spec liste déroulante simple et non multiple
assets/vue/components/signalement-list/TheSignalementAppList.vue
Outdated
Show resolved
Hide resolved
assets/vue/components/signalement-list/components/SignalementListFilters.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.
2 commentaires de code. Mais tests concluants 👍
Tests concluants ! |
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.
Juste une incompréhension, mais tests ok
assets/vue/components/signalement-list/services/badgeFilterLabelBuilder.ts
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.
Tests ok, mais une petit contrôle de sécu à rajouter
|
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 tout bon
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.
Tout bon pour moi aussi
Ticket
#2597
Description
Quand on commence par un tri alphabétique croissant, le tri alphabétique décroissant reste en croissant, et vice-versa, le tri n'est plus pris en compte quand on ajoute un filtre
Changer le tri "plus récent"/"plus ancien" en utilisant la date de dernier suivi plutôt que la date de dépôt (qui est redondant avec la référence)
Changements apportés
territoires[]=13
parterritoire=13
)Pré-requis
Les données de liste déroulante provenant de la base de données sont en cache, si besoin
Pas de date de dernier suivi sur la carte, préparer la requête SQL suivante
Si besoin de toutes les EPCIs (il y'en as quelque uns dans les fixtures)
make console app="load-epci"
Tests