Open victorsalaun opened 7 years ago
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
Aaaah je crois que je vois... src/app/lib/ :(
@Lucas-C yes I think it was about this folder
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
Je reporte le bilan fait dans RTB fin octobre dernier, priorisé "à la louche" par moi du plus au moins critique :
bower
qui est déprécié : https://bower.io/blog/2017/how-to-migrate-away-from-bower/eslint
) en placeproperties.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 codehttp-server-legacy
par http-server
et établir si on utilise yarn
+ yan.lock
ou npm
+ package-lock.json
cf. Dockerfile#L8addFromModel
ng-switch
dans event-modal.html et les directives associées, très redondantesconfig.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
eslint
:
"plugins": [... "promise"],
"extends": [... "plugin:promise/recommended"],
.to_rest_entity
-> utiliser Restangular ?grep -lF 'type="submit" id="submit" value="Submit" ng-hide="true"' $(git ls-files src/app)
.eslintrc
The project's code need refactoring: