Closed azarz closed 3 years ago
@elias75015 @lowzonenose please review!
Best review ever ever.
Aucun soucis remarqué à part le triple clic pour terminer le tracé qui casse l'affichage (d3 et default) : non lié à cette PR
Ok, ben faudra regarder ça et creuser un peu avant l'intégration au portail...
Pull request checklist
Verifiez que votre Pull Request remplit les conditions suivantes :
npm run build
) a été lancé localement et s'est correctement déroulénpm run samples
) ont été testés et validés localementnpm run test
) sont passés localementType de Pull request
Quel type de changement cette Pull Request introduit-elle : - [x] Bugfix - [ ] Feature - [ ] Mise à jour du style du code (syntaxe, renommage de fonctions) - [ ] Refactoring (lisibilité/performance du code, sans changements fonctionnels) - [ ] Changement sur le processus de build - [ ] Contenu de la documentation - [ ] Autres (décrire ci-après) : ## Quel est le comportement actuel (avant PR) : La précédente PR sur le profil alti a cassé les profils avec amCharts, et abîmé ceux avec D3. Avec cette branche, résolution des problèmes introduits (seul code modifié : dans les fonstions spécifiques à amCharts et D3) Numéro du ticket : N/A ## Cette PR introduit-elle des breaking changes ? - [ ] Oui - [x] NonAutres informations
Bug intrdoduit :