search

jvanoost / MMM-SNCF

A magic mirror module which retrieve data from SNCF API
MIT License
6 stars 9 forks source link

Amélioration affichage + options #5

Closed abrochet closed 4 years ago

abrochet commented 4 years ago

Bonjour @jvanoost,

Merci pour le module, c'est une super idée ! 👍 J'ai apporté des améliorations sur l'affichage et j'ai modifié certaines options. Si vous utilisez encore votre module, je souhaite vous proposer mes modifications, si jamais ça vous intéresse bien évidement... :)

A bientôt, Anthony,

jvanoost commented 4 years ago

Bonjour ! Merci pour la contribution !

N'étant pas un champion du monde git, comment puis-je tester tes propositions ? Je sais qu'il existe qq utilisateurs, il faut juste faire gaffe que si on fait pousse les changements ça ne casse pas le fonctionnement après la mise à jour puisqu'il faut du coup faire évoluer le fichier config.js

Si j'ai bien compris, tu as renommé les login / password (où j'utilisais que le login) pour mettre APIKey, ce qui est finalement assez logique ;)

Je l'utilise tous les jours chez moi, je suis bien sur preneur de toute amélioration. C'est le but de tout code libre non ??

Bonne journée

abrochet commented 4 years ago

Bonjour,

Il est vrai que je n'ai pas pensé à la rétrocompatibilité... (J'ai anglicisé les variables pour coller au reste du code).

Je n'ai également pas trop de connaissance en git, au travail on utilise SVN mais j'ai trouvé ça sur internet : https://www.christopheducamp.com/2013/12/16/forker-un-repo-github/

Vous voulez que je fasse en sorte que les anciens paramètres soit toujours valide ?

Je vais encore apporter une modification, de temps en temps il y a un plantage "non visible" car l'API de la SNCF ne retourne pas toujours une date valide :

[09:41:33.999] [WARN] Deprecation warning: value provided is not in a recognized RFC2822 or ISO format. moment construction falls back to js Date(), which is not reliable across all browsers and versions. Non RFC2822/ISO date formats $ Arguments: [0] _isAMomentObject: true, _isUTC: false, _useUTC: false, _l: undefined, _i: undefined, _f: undefined, _strict: undefined, _locale: [object Object] Error at Function.createFromInputFallback (/home/pi/MagicMirror/node_modules/moment/moment.js:320:98) at configFromString (/home/pi/MagicMirror/node_modules/moment/moment.js:2385:15) at configFromInput (/home/pi/MagicMirror/node_modules/moment/moment.js:2611:13) at prepareConfig (/home/pi/MagicMirror/node_modules/moment/moment.js:2594:13) at createFromConfig (/home/pi/MagicMirror/node_modules/moment/moment.js:2561:44) at createLocalOrUTC (/home/pi/MagicMirror/node_modules/moment/moment.js:2648:16) at createLocal (/home/pi/MagicMirror/node_modules/moment/moment.js:2652:16) at hooks (/home/pi/MagicMirror/node_modules/moment/moment.js:12:29) at Class.processTransports (/home/pi/MagicMirror/modules/MMM-SNCF/node_helper.js:160:57) at /home/pi/MagicMirror/modules/MMM-SNCF/node_helper.js:44:22

Bonne journée,

jvanoost commented 4 years ago

Si vous pouvez juste ajuster pour que les variables qui sont utilisées dans le config.js fonctionnent sans avoir à faire de modif. Pour le reste, j'ai vu le rendu sur votre github c'est top ! Dès que c'est ok de votre coté faites moi signe et je fais le merge du PR.

jvanoost commented 4 years ago

Au top ! merci ! Je fais le merge !