Web App created with Nextjs, rest countries Api

Community feedback

• 

  Paweł Pohl•390

  @Pawel1894Posted over 1 year ago

  Hey, your app is really good looking!

  Some issues i found:

  • filter by region is not working
  • never user react context for fetching data like this it makes your app super slow.
  • Take advantage of next.js features - countries data does not change frequently so you can use getStaticProps to prerender them on the build time and make your app loading smoothly.
  • search for a country works much better without sending api, filtering list of object with length of ~250 is much faster so I would recommend doing this on client side

  You can check out my solution i also made it in next.js and i got 30ms loading for a homepage.

  1

  ---

  

  Silver Idahosa•170

  @SilverBeingPosted over 1 year ago

  @Pawel1894 Thank you so much for taking out your time to do this. I would try to implement your suggestions. If you would help me check my codes. I actually tried getStaticProps approach. it wasn't working so I commented it out. If you could help me check what was wrong. It would mean alot. But are you sure the filter by region doesn't work. I just checked, it works perfectly. Thanks again.

  1

  

  Paweł Pohl•390

  @Pawel1894Posted over 1 year ago

  @SilverBeing Oh, I'm sorry then. I think restcountries api has some server issues right now and it just takes super long to even load into their website :P It must be working for you fast because you have cached it in browser already :D

  0
• 

  Emmanuel Daniel•1,170

  @coderdanniePosted over 1 year ago

  Hi. Congrats on the completion of this project. An extra features you can add is to store the themes state using browser localstorage so that your site can maintain the theme state even if the page was reload. Right now if I switch to light mode and reload the web page its switching back to the default theme. Using browser storage to store the theme and accessing the themes based on user preference will prevent this from happening. https://developer.mozilla.org/en-US/docs/Web/API/Window/localStorage Also, you can conditionally render the border countries meaning if perhaps there is a country without border countries like cape verde, you can include something like no border countries

  0