-
Notifications
You must be signed in to change notification settings - Fork 7
Choix techniques
Loup Theron edited this page Feb 22, 2024
·
4 revisions
- On n’utilise pas vraiment
/fp
car nous n'avons aucune utilisation deflow()
fp - On n’inverse pas les params dans tous nos utils
monitor-ui
& utils internes
De ce fait, il vaut mieux rester sur lodash
, mais :
- On utilise
map
pour les listes avec du lodash qui retourne une fonction pure (et pas de_.chain()
) - Si on doit chaîner sans listes, on prévilégie des variables intermédiaires
- Nommage :
- On nomme en language métier si c’est un composant “smart”
i.eonAlertChange
au lieu deonChange
- Si c’est un composant partagé, on le nomme de façon générique
i.edata
au lieu dereportings
- Optimisation React :
- useMemo & useCallback
Vérifier le ratio : quand est-ce qu’on optimise avec une mémoization ?
On fait un useCallback quand on passe une fonction en props
- Dépendances :
- On parle de cet ajout dans l’équipe avant de le faire
- On utilise lodash plutôt que ramda pour les utils
- TODO: Regarder comment on peut faire un uniqBy dans un reduce
- On utilise dayjs
- On passe datefns en dayjs quand on passe dessus
- Routing
- On met à jour react-router (Ivan fini la MAJ de la deps)
- On utilise un pattern URI-like qu’on stocke dans redux et on appelle
matchPatch
pour vérifier si un composant doit s’afficher
- CSS global
- On passe en composant MonitorUI les composants rsuite de Fish
- une fois que c’est fait, on pourra enlever le rsuite-override.css et le wrapper NoRsuiteOverride
TODO Changer le modèle de PR pour faire tous les changements petit à petit
- Plan de reprise d’activité
- Rajouter dans readthedoc la doc de Thomas
- Sur Fish, faire un fichier de migration pour les MEPs
- Zabbix
Règles à implémenter :
- Si mem inf à X MB, on restart le conteneur
- Avoir un graph de consommation par container
- Avoir l’état du disque
- taille de la db
- taille de la sauvegarde