Correct a typo

[goulvenclech]

Bonjour,

Dans le fichier data.js on trouve une coquille à la ligne 24 : todayScore devient score pour l'user 18.

Je sais que les projets d'OpenClassrooms sont connus (du moins entre étudiants) pour nous fournir des JSON et des codes sources de mauvaises qualités, généralement sur les Discord d'étudiants on se partage d'ailleurs les différentes corrections (orthographiques ou typos) à faire pour les corriger. Et ce back end ne déroge pas à la règle, avec une structure de données inutilement complexe (exemple : kind dans le end point performance) et une nomenclature aléatoire (exemple : id ou userId en fonction des endpoints) qui je suppose sert à nous mettre au défi de gérer des Api mal structurées.

Mais sur le projet 12 une vraie erreur est génante, car nous ne sommes pas censé toucher au back end, seulement développer l'application front end SportSee qui fetchera ce back. Avoir une erreur m'empêche donc de simplement indiquer sur mon projet "Pour faire fonctionner le projet, merci de cloner le répo du back end et de le lancer avec yarn start" et ne pouvant pas décémment demander à un visiteur de corriger lui même l'erreur quand il clone le répo, je risque d'être obligé de simuler le back end avec des fichiers JSON dans mon code source.

Je pense donc que cette simple Pull Request permettrait de supprimer cette friction, et de permettre d'avoir un back end qui fonctionne. Ce qui serait le minimum vu qu'il ne contient que deux utilisateurs enregistrés !!!

En vous remerciant.

[najamelan]

@tdimnet Je ne veux pas trop ajouter au cynisme, mais peut-être dans un an ou deux on peut tester le bouton vert qui dit Merge Pull Request en bas de cette page? Question d'avoir une génération d'élèves qui n'est pas obligé de passer par la un jour?

[tdimnet]

@Najamelan : Mhhhh, c'est pas du tout mais alors pas du tout agressif comme message, j'aime beaucoup !

Alors, premier élément, je ne sais pas quel est votre niveau sur github mais si personne ne me ping ou du moins ne me demande des reviews une PR, je ne suis pas forcément au courant qu'il y a une.

Deuxième élément, c'est OC ici qui a les droits, comme dans de nombreux repos du Student Center. Du coup, ça veut dire qu'il faut passer par un cycle de faire une demande à OC pour mettre à jour le projet. Mais encore une fois, si personne me ping... C'est beau le cynisme hein mais c'est pas forcément ça qui va vous faire modifier le projet ^^.

J'avoue que je pourrais faire ça mais étant donné la teneur de votre message, ça me laisse un peu perplexe.

[najamelan]

Je veux bien clarifier qq trucs. Ce n'est pas a cause de toi qu'on en deviens cynique. Je t'ai ping parce que j'ai l'impression que c'est toi qui a introduit l'erreur, et je ne vois pas un compte OC que je peux ping. Puis pourquoi la teneur:

1. Comme GoulvenC l'a déjà fait remarquer, dans toute la formation il n'y a pas un projet correct. Le code des projet ne passe pas un standard professionnel, laisse tomber qu'il soit exemplaire, comme ça devrait être le cas de la part d'enseignants. Au bout d'un moment ça use la patience.
2. Ok, tu publie du code publique avec des bugs, sur lequel d'autres vont dépendre mais t'est pas souscrit au dépot. Dac mais pas seulement ça, personne d'autre de OC à l'air de l'être non plus. Car des simples PR comme celui-ci traînent pendant des mois. Je ne connais pas ton role chez OC, ni le processus de OC. Je constate juste que tout ça c'est quand même étonnant pour une école.
3. Surement je passe juste une mauvaise journée car le code d'OC ce n'est pas le seul qui bugge.
4. Seulement celui qui ouvre le PR peut demander des reviewers je pense.

Du coup désolé pour le ton, ne prends pas personellement.

[tdimnet]

J’ai envoyé un e-mail à OC pour les prévenir. Je vais aussi me replonger un peu dans le projet pour te fournir une réponse complète.

Concernant la structure de données inutilement complexe, c’est voulu. D’une part, parce que vous allez travailler avec des API comme ça dans votre futur taff. D’autre part, parce que cela vous force à faire de la modélisation de données (via des Constructor Pattern).

Je vais regarder la partie Id/userId. Il peut très bien s’agir d’une coquille. Encore une fois le code a bientôt 2 ans et oui, ça m’arrive de faire des fautes et des bugs côté code 😅. Et ça va continuer à m’arriver.

Je ne travaille pas chez OC mais ça ne va pas m’empêcher de pousser la modification (si elle fait sens) et de modifier le projet. Il faut juste me le demander poliment (j’attends pas 15 Smileys différents, une mention sur la PR en me demandant ce que j’en pensais aurait été suffisant).

[tdimnet]

Je fais un petit up dessus : j'ai envoyé un mail à OC pour en parler le 28 février et qu'on fasse la modif. On est le 18 mai, je n'ai toujours aucun réponse.

Voilà, voilà 😅

[najamelan]

Merci d'avoir fait du suivi. Je vais essayer de mon coté de faire des retour à OC car ma formation fini tantot. Je vais leur suggérer d'avoir au moins un employé(e) qui soit souscrit au dépots github et qui s'en occupe des erreurs, des issues et des PR.

[Jaucordi1]

Plus d'1 an après, erreur toujours présente, ce projet est d'ailleurs utilisé en tant que P12 dans d'autres parcours… Serait-ce une typo souhaitée ou sont-ils définitivement incompétents ?

Quelqu'un a eu des nouvelles d'OC à ce sujet ?

[tdimnet]

@Jaucordi1 : non, toujours pas (mais ça m'étonne assez peu, malheureusement...)