@MartineauRemi
Posted
Hey Roy ! You did a nice job completing this challenge, congratulations :) The js part is good, but could be improved a bit :
-
You used Promises but didn't cover the eventuality of a failure coming from the API. So you could add a 'catch' clause to handle errors, and display a custom error message for example. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Using_promises
-
Your js file is small for now, but its size may increase over the time. I suggest you take the code inside of 'then' and create a separate function, that you call in your 'then'. Your code will be easier to read (especially for other developers working with you). This advice is purely personal though. Others will tell you there is no need to do that.
-
This one is a detail, but I think you forgot to remove a console.log line 28.
Hope this clear enough and that it can help you. Keep up the good work, and enjoy your #100daysOfCode challenge :D
