feat: revue de l'écran paramètres #3270
Conversation
totakoko
force-pushed
the
feature_revue_ecran_parametres_of
branch
from
1595a4a to
da44e8a
Compare
feat: configuration ERP v2 feat: wip configuration v3 feat: enregistrement ERP non supporté feat: sauvegarde l'auteur de la configuration ERP feat: redirection apiv3 ok
totakoko
force-pushed
the
feature_revue_ecran_parametres_of
branch
from
e81b6b6 to
b858cdc
Compare
totakoko
changed the title
| @@ -56,6 +56,7 @@ | |||
| "recoil": "0.7.7", | |||
| "remixicon": "2.5.0", | |||
| "shared": "workspace:*", | |||
| "usehooks-ts": "^2.9.1", | |||
There was a problem hiding this comment.
Mieux que maison en effet !
There was a problem hiding this comment.
De base, je ne suis pas forcément pour ajouter beaucoup de dépendances. Parfois, je préfère avoir un truc maison dont je sais qu'il marche bien plutôt qu'un truc externe, qu'il faudra mettre à jour etc.
Mais là, c'est l'inverse 🙃
Co-authored-by: Raphaël Huchet <[email protected]>
| id: string; | ||
| name: string; | ||
| helpFilePath?: string; | ||
| helpFileSize?: string; |
There was a problem hiding this comment.
Est-ce vraiment utilisé ?
There was a problem hiding this comment.
Tu parles des helpFile* ? Oui pour la page de configuration ERP v2
| @@ -20231,6 +20231,7 @@ request-debug@latest: | |||
| remixicon: 2.5.0 | |||
| shared: "workspace:*" | |||
| typescript: 5.0.4 | |||
| usehooks-ts: ^2.9.1 | |||
There was a problem hiding this comment.
C'est quoi la valeur ajoutée de cette lib ?
There was a problem hiding this comment.
C'était pour avoir un hook useLocalStorage qui ait une bonne interface et du typage plutôt que d'adapter le hook existant.
🚀 Prévisualisation |
|
🎉 This PR is included in version 3.62.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
| @@ -165,7 +165,7 @@ function EffectifsPage(props: EffectifsPageProps) { | |||
|
|
|||
| <Box mt={10} mb={16}> | |||
| {Object.entries<any[]>(effectifsByAnneeScolaire).map(([anneeScolaire, effectifs]) => { | |||
| if (anneScolaire !== "all" && anneScolaire !== anneeScolaire) { | |||
| if (anneeScolaire !== "all" && anneeScolaire !== anneeScolaire) { | |||
There was a problem hiding this comment.
J'ai trouvé la régression @rap2hpoutre @sbenfares.... Comme quoi les fautes d'orthographe font partie des bonnes pratiques chez certains développeurs. 🙃
Résumé :
Reste à faire :
Peut-être refacto page Paramètres=> pas forcément en fait