Random Citation With Rolling Dice

Community feedback

• 

  Elaine•11,400

  @elaineleungPosted about 2 years ago

  Hi Lucas, great work building this out with React, and I like your the various transition effects you added 😊

  What you did as the loading screen is a very interesting choice and probably the first one I've seen that does that. It makes me think that you can even write as an advice "The first advice is to roll the dice" 😂

  Aside from that, the main suggestion I have here is to add error handling for your async/await, which you can do using try/catch. About the delay, it may have to do with caching; also right now there's an issue where on Firefox (which I use), the next advice is not able to load, and I've seen this on a number of solutions. You can try adding a header object in fetch, like this:

  fetch("https://api.adviceslip.com/advice", { cache: 'no-cache' })
  

  You can test it out on Firefox and see if you have the browser. Well done here, and keep going 😊

  Marked as helpful

  1

  ---

  

  Lucas Matheus•400

  @LukiticasPosted about 2 years ago

  @elaineleung thank you so much, Elaine!! I'll try to address these issues asap ♥️. About the caching, i thought it was a API problem and that i couldn't fix it directly.

  1