Aman Singh Bhogal• 1,010
@asbhogal
Posted
Hi Motunrayo,
Great job! The app functions well and matches the mockups nicely
Just a few things I think are worth mentioning here:
- I've looked at your JS code, and you should ideally have a
catch()
method in your API call to catch any errors that may be thrown during the function parse. A commoncatch()
block, added underneath thethen() method
, includes logging this error to the console to help with debugging. Here's a good article by MDN explaining this (https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API/Using_Fetch#checking_that_the_fetch_was_successful) - Your
button
should be an element of its own and not nested in adiv.
This makes it inaccessible to keyboard users. I tried accessing it using mine, and wasn't able to. - It might be worth also formatting some of your code so its easier to read (some of your end parenthesis aren't indented, which can make it difficult for other developers to follow your code.) If you use VS Code, the Prettier extension can automatically do this for you. Here's a link to it Link
- You should locally host your Google Fonts for privacy and performance reasons. Here's a good link explaining why and how to do this Link
Hope this helps!
1