@milimyname
Submitted
It was a long journey to make it work even though it isn't the hardest challenge here but I retook some lectures from the course of Jonas Schmedtmann. I hope It'll look responsive in your device!
@omonteon
@milimyname
Submitted
It was a long journey to make it work even though it isn't the hardest challenge here but I retook some lectures from the course of Jonas Schmedtmann. I hope It'll look responsive in your device!
Hey Mili,
Congrats on finishing the challenge, I just submitted this one a few days ago too. Below I share my findings, I hope these are helpful :)
Highlights:
Possible improvements:
Marked as helpful
@richardcyrus
Submitted
Feedback welcome. Uses React Query and Axios to handle the data, search and filtering. Also uses Cypress (first time using) for some basic tests.
Hey Richard !
This is my first time providing feedback in the community, I hope my comments are helpful.
First of all, in my opinion, all the requirements are met both in desktop and in mobile. Congrats on that !
However, I did found some possible issues:
There is a 404 when erasing the search text. If you write something in the search input and then you erase all of it, multiple requests are sent with 404 response status code. After a few of these requests fail, a message error "Request failed with status code 404" is displayed. A possible alternative I would suggest is handling this by requesting all countries, same as the request when the app loads.
There is a quick flicker when you have selected the dark mode and you refresh the page. I solved this problem by using a script that handled setting the default theme before loading the app bundle. https://dev.to/ayc0/light-dark-mode-avoid-flickering-on-reload-1567
If you run an audit using chrome dev tools, some performance issues are shown. I think it might be related with the images. Maybe that was not your focus with this solution, just something I noticed that I thought it might help.
That's it, I hope this is useful for your future learning.
BTW I think it's cool that you started using cypress, it's one of my favorite testing tools.
Marked as helpful