[Signalements] Afficher la liste des signalements depuis un sémaphore avec un filtre de un mois

[claire2212]

• Modification suite au retour de Manon : Ajouter "voir les signalements" sur les fiches sémaphore #1832 (comment)
• Remplacement suppression des as any

Related Pull Requests & Issues

• Resolve Ajouter "voir les signalements" sur les fiches sémaphore #1832

---

• Tests E2E (Cypress)

[ivangabriele]

Mes petites note ^^ :

• J'ai vu qu'il y a plusieurs optionValueKey= ... as any/string, je pense que beaucoup si ce n'est tous sont remplaçable par une simple string sans typage.
• Dans le mesure du possible c'est à mon avis beaucoup plus solide et utile de typer en déclaratif plutôt qu'en "transformatif", en clair utiliser les const myConst: MyType, MyClass<> et satisfies plutôt que d'utiliser des as qui sont à peine type-checkés et garantissent donc peu le typage.
• J'ai aussi tendance à trouver ça plus clean de bien typer tout ce qui est initialisé en constante ou passé en param pour ne pas avoir à surtyper ce qui est intermédiaire via des as/satisfies ou via des génériques passés explicitement. Mais ça je sais que tu (et sûrement Maxime) penses déjà la même chose !

Je garde en tête qu'on utilise TS pour à la fois limiter les erreurs possibles et documenter ce qu'on attend.

NB : Et je suis par avance d'accord avec l'argument que d'écrire const myVar = [] as MyType[] donne le même résultat que const myVar: MyType[] = [], c'est plus une remarque de goût/consistance avec le fait que as is evil dans d'autres cas 😁 (et je n'ai moi-même introduit les satisfies qu'il y a quelques mois).

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud