Corrige le probleme de 404 sur la version disponible

NextDom / nextdom-core

NextDom Core >>

https://www.nextdom.org/

GNU General Public License v3.0

21 stars 13 forks source link

Corrige le probleme de 404 sur la version disponible #1635

Closed ralmn closed 4 years ago

ralmn commented 4 years ago

Fix #1567

Sylvaner commented 4 years ago

Bonjour, Merci pour ta correction, juste quelques petits changements

Nous essayons de remplacer toutes les chaines de caractères par une variable, c'est pour ça que nous mettons Common::CORE plutôt que simplement 'core', cela peut paraitre compliqué pour ce cas mais il y a un réel plus sur la vision globale
Si tu peux, nous avons oublier de remplacer ce \com_http, si tu veux tu peux le changer en ComHttp et donc rajouter 'use NextDom\Com\ComHttp;' en haut du fichier.

Maintenant que t'as mis le doit dessus, il va falloir comprendre comment on a pu passer à côté :)

ralmn commented 4 years ago

Merci pour la review, pour le Common::Core j'ai fait une erreur de copier collé depuis la master :( J'ai aussi fait le changement le ComHttp

Sylvaner commented 4 years ago

je crois que pour la version il faut que tu vises /data/config/Nextdom_version

???

Perso je valide, j'ai testé et ça fonctionne :)

sonarcloud[bot] commented 4 years ago

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
0 Code Smells

No Coverage information
0.0% Duplication

ralmn commented 4 years ago

(désolé j'ai missclick le bouton de demande de re-review)

je crois que pour la version il faut que tu vises /data/config/Nextdom_version

Hum le /data/config/Nextdom_version je le trouve pas dans le repo. Le seul que j'ai trouvé Nextdom_version se trouve dans /assets/config/Nextdom_version

Et si non je viens de corriger le problème du CHANGELOG.md

Gis70 commented 4 years ago

Rejoins nous sur Discord https://discord.gg/aXVQCbN @ralmn nous cherchons des Beta Testeur et Dev