Advice Generator App using Fetch API(async-await) and Media Query

Community feedback

• 

  David•7,960

  @DavidMorgade

  Posted almost 3 years ago

  Hey Evbowe, congrats on finishing the challenge! pretty good job, your API calls are working properly

  Regarding your question, yes, you should try using a try catch block in your functions so if the API fails to load the data it will give you an error that will get catched, then with this error you can for example log it into the console, or display it on the screen, or return a error state... Million of posibilities.

  Apart from that, try using a native <button> for the dice instead of a <div> it fits much better the purpouse, you could also add the event listener to the entire button, because right now is only working on the dice image.

  Hope my feedback helps you! if you have any questions, don't hesitate to ask! great work

  12

  Marked as helpful