Skip to content
Submitted almost 2 years ago

Advice Generator (Flexbox+API)

accessibility, fetch, lighthouse
Abere Fejiro•490
@Fejiro001
A solution to the Advice generator app challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Hi this is Fejiro, and this is my solution to the Advice Generator

  • Got it as close to the design
  • Made sure it was responsive
  • Achieved perfect 100 score on lighthouse

Any suggestions on how I can improve are welcomed

Code

Please log in to post a comment

Log in

Community feedback

  • Sajana Wijesinghe•490
    @StormKing969

    Not sure if you noticed but you seem to be getting duplicate responses from your API call. to prevent that, you'll have to disable caching. Also I'd suggest that you put your call in a try catch methods. That way, you can log any error you get.

  • JJacobPR•240
    @JJacobPR

    Hey,

    I love your dice animation. What I would improve is the random data generation. As this API tends to get a little bit sloppy, you can use the documentation to find route for a specific advice and generate the ids yourself in frontend to ensure that user would get random advice. I know that normally it would be backend's fault but here we are just practicing, aren't we 😀

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord