REST Countries API with Color Theme Switcher with Vue & TailwindCSS

Community feedback

• 

  Raza Abbas•790

  @RazaAbbas62Posted 10 months ago

  Hi, it looks good, but without the repo URL, I couldn't check your code.

  One suggestion is to handle cases where a country has no information like borders or other details, and display suitable information in those cases. For example, Antarctica doesn't have such detailed information, and the page shows a loading spinner. I had a similar issue with mine but realized that later

  Enjoy coding :)

  1

  ---

  

  Abi•300

  @abimh66Posted 10 months ago

  Thanks for the feedback, here is the repo https://github.com/abimh66/rest-countries @FazeenIjaz

  1

  

  Raza Abbas•790

  @RazaAbbas62Posted 10 months ago

  @abimh66

  Your code is well-structured. Consider improving error handling by displaying user-friendly messages. Add comments for better readability. Ensure consistent formatting, provide clearer loading feedback. Validate data existence before accessing properties using optional chaining and nullish coalescing. Remove any unused imports for a cleaner codebase. Consider optimizing API requests, like batching border country requests, for better performance.

  1
• 

  Petrit Nuredini•2,860

  @petritnurediniPosted 10 months ago

  Hello mate, congrats on finishing your challenge!

  I wanted to review your code but it appears that you are missing the repository url. Can you check it again and hopefully I can give you a review for your solution?

  Best regards.

  1

  ---

  

  Abi•300

  @abimh66Posted 10 months ago

  here's my repo https://github.com/abimh66/rest-countries @petritnuredini

  0