@AlekseiBestuzhev
Posted
Hello.
- Try to open China and you will see the error. This is due to the fact that you are using the search by name. From the documentation: "Search by country name. If you want to get an exact match, use the next endpoint. It can be the common or official value". China, Macau, Hong Kong and Taiwan have the match in the official name. The same situation with South Georgia and Georgia.
- When I open Antarctica, the app crashes. Check the console.
- Not all properties exist in all countries. The capital of Antarctica, for example. You need to handle their absence. I think the error is being thrown for the same reason.
- In filtering by regions, there is no way to reset the filter. I also advise not to make a new request after selecting a region, but to filter the received ones.
- Missing region Antarctica.
- I would make the request to the server separate from the components.
- Easy to read project structure. Everything is very clear and easy to navigate.
👏 Well done for completing this challenging project. Good luck!
Marked as helpful
@jCasia
Posted
@AlekseiBestuzhev Thank you so much Aleksei😍 I really appreciate it! It was a tough challenge for me haha, But yes, some of the things you pointed out, I have failed to notice! I will definitely take note of everything you said!
Once again thank you!😊
@AlekseiBestuzhev
Posted
@jCasia Difficulties make us stronger! 💪
By the way, for regions use filtering and for the search bar try useDebounce. Useful hook for similar situations.
Marked as helpful
@jCasia
Posted
@AlekseiBestuzhev haha true true🥹, I'm still weak though but I'm slowly getting there haha and Ooohh I've never come across that hook before, looking at the docs for it now and its definitely a good thing to include in a search bar!