Dette technique

[victorsalaun]

The project's code need refactoring:

• remove hard coded lib
• externalize CSS / fonts / images from app directory
• use AngularJS specialised files (controller, directive, factory, module, config...) instead of one single js file per functionality
• lint code (e. g. with jshint)

[Lucas-C]

In case you remember @victorsalaun, what did you mean by "remove hard coded lib" ? :)
All of the <script> tags in index.html ? cf. https://github.com/voyages-sncf-technologies/hesperides-gui/blob/3.0.3/src/app/index.html#L37

[Lucas-C]

Aaaah je crois que je vois... src/app/lib/ :(

[victorsalaun]

@Lucas-C yes I think it was about this folder

[Lucas-C]

cf. CR dans RTB pour une rapide étude de la dette technique sur ce projet, complémentant ce qui a déjà été listé ici

Lorsqu'on adressera le sujet, il faudra découper cette issue en plusieurs sous-tâches

[Lucas-C]

Je reporte le bilan fait dans RTB fin octobre dernier, priorisé "à la louche" par moi du plus au moins critique :

• rétablir tests e2e d’intégration avec le backend
• mesurer la couverture de code
• utilisation de bower qui est déprécié : https://bower.io/blog/2017/how-to-migrate-away-from-bower/
• gestion des libs JS versionnées dans le repo git dans src/app/lib + libs vieilles, pas à jour, ex: angular-material
• utilisation de https://github.com/stidiovip/hesperides-datetime-picker sur lequel nous n'avons pas les droits
• pas de linter (ex : eslint) en place
• CSS non découpé en composants, et aucune convention de nommage en place (OOCSS / SMACSS / BEM)
• plus long fichier source : >2500 lignes de code JS (properties.js) -> solution : sortir les factory Angular de ce fichier (et même faire ça globalement) -> rétablir et se baser sur le check eslint "angular/file-name" pour mieux découper le code
• gestion i18n à revoir pour les erreurs backend
• certaines parties sont désormais inutile et devrait être supprimées : Nexus / getNdl + Swagger
• utilisation de jQuery en parallèle d'Angular, et de lodash + underscore ???
• nous avons 3 versions mineures de retard sur Angular JS
• remplacer la dépendance http-server-legacy par http-server et établir si on utilise yarn + yan.lock ou npm + package-lock.json cf. Dockerfile#L8
• fonction globale addFromModel
• supprimer le ng-switch dans event-modal.html et les directives associées, très redondantes
• supprimer le config.json par souci de simplification et d'uniformisation : tout les valeurs de configuration pourraient être injectées par l'entrypoint Docker directement dans index.html
• nettoyer notre utilisation des promesses JS en activant ce plugin eslint:

    "plugins": [... "promise"],
    "extends": [... "plugin:promise/recommended"],

• avoir un fichier JS dédié par contrôleur / factory
• il y a beaucoup de code boilerplate dans les fonctions .to_rest_entity -> utiliser Restangular ?
• boutons de soumission invisibles : grep -lF 'type="submit" id="submit" value="Submit" ng-hide="true"' $(git ls-files src/app)
• supprimer les fichiers non utilisés (notamment certains fichiers html)
• tout passer en camelCase
• supprimer certaines exclusions dans .eslintrc