Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/fetch api #2

Merged
merged 45 commits into from
Aug 1, 2023
Merged

Feature/fetch api #2

merged 45 commits into from
Aug 1, 2023

Conversation

ClaudiaRojasSoto
Copy link
Owner

Description:

This pull request implements the functionality to connect with the Rest Countries API and display country data on the Details page.

The following changes have been made:

  • The Axios package has been added to handle HTTP requests to the API.
  • A new React component called "Countries" has been created to manage the retrieval and display of country data.
  • State management using React Hooks (useState and useEffect) has been implemented in the "Countries" component to handle country data and make the API call when the component mounts.
  • The data is displayed on the Details page when a user selects a country from the list.
  • The Gitflow best practice was followed, with a dedicated branch for this new functionality.
  • No linter errors were detected, and JavaScript best practices were adhered to.
  • The code documentation has been done professionally, and a detailed description of the changes in this pull request has been added.
  • The design, unit, and integration testing, implementation of filtering functionality, and other project requirements will be addressed in future pull requests.

Please review the changes and let me know your comments.

…age to the source folder with the images of the continents and the flags of america and europe
…ow the continents on the flags page and to show the name with the first letter on capital in the same page
…raslate the folder inside the reducer folder
…ick function to handle the clic event on the countries
…first play obtaining the lat and lon from the country select by the flag on the first api
@ClaudiaRojasSoto ClaudiaRojasSoto merged commit 3c5fbfa into develop Aug 1, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant