Second Attempt at REST Countries API. App now uses only 1 API call.
Design comparison
Solution retrospective
This is my second submission for this challenge.
While I fulfilled the brief for the project, I wanted to give myself the additional challenge of fulfilling the requirements using a single call to the API.
Rather than calling the API multiple times for fetching country details and when navigating back to the main list of countries. I wanted to have one persistent piece of state that could manage everything.
In the previous submission, I found myself dealing with bad request handling. After a number of calls - potentially down to API call throttling. I found a single call to the API satisfied everything I needed to do. It's made the initial load a little longer, but the navigation and use of the app, very fast afterwards. I'll probably filter the fields in each API request later.
In my next submission, I'll look at component-ising things more!
Any questions or improvements, let me know!
Community feedback
- @Dark-LoverPosted almost 3 years ago
i like your solution
what u said is what everyone should do, minimize as possible api calls, cuz in real world projects more requests means paying more money. and in this project a single call is enough, u can then filter from it all the data u need in every component.
keep coding
Marked as helpful0@gcarter89Posted almost 3 years ago@Dark-Lover Thanks! I really appreciate it.
Exactly! It also makes error handling across the application much easier to manage, and the performance is much better between renders of the main page and my detail page since they're drawing from the same call!
0 - @EmmanuelHexerPosted almost 3 years ago
Great job overall man. Keep it up
- One tip i can give you is to always read the reports at it will help you know the issues in your code ,so that you fix it.
Marked as helpful0@gcarter89Posted almost 3 years ago@Phalcin Hey Emmanuel, thanks for the feedback! Really appreciated.
7 of the 8 html issues seem to be related to the build step of my app, so I can't fix them. I did fix the h2 in the button element, though!
0 - @MatiX221Posted almost 3 years ago
Firstly you should read reports.
Secondly you can't unselect continent filter, so if you select europe you can't set it to all.
And thirdly if you have something in search bar and then change region in continent filter. It will show all countries from that continent and not country that you searched from that continent.
0@gcarter89Posted almost 3 years ago@MatiX221 Hi MatiX. Thanks for the feedback.
With the reports, the css and the html info seem to be related to the build. They're not things I can correct unfortunately.
I did fix the issue with the h2 element in the button, though.
Everything else you suggested, I've just added to the site! Hope the code is useful!
0
Please log in to post a comment
Log in with GitHub
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