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] Implement Data Fetching from API #3

Merged
merged 8 commits into from
Apr 3, 2024
Merged

Conversation

Vanessa-Kris
Copy link
Member

  • Utilized mockAPI data
  • Made mobile responsive
  • Updated theme

…rom API"

This commit includes the following changes:
- Updates to several dependencies, including D3, React Router, and React Simple Maps.
- Addition of new components such as Loader, Help, and Contact.
- Addition of new pages such as Help and Contact.
- Fetching data from an API and storing it in state.
- Modification of existing components to use the new data and components.
- Addition of a new route for the Data page.
- Modification of the TheTable component to accept a new prop for isLoading.
- Addition of a new function to handle downloading data as a CSV file.
- Modification of the Dashboard component to display the total number of entries and to use the new handleDownload function.
- Modification of the TableDialog component to use the new testData prop and to display the new columns.
- Addition of a new function to handle data fetching in the Utils folder.
- Removal of unused imports and console.log statements.
- Addition of a new index.css file and removal of unused CSS.
- Modification of the yarn.lock file due to the updated dependencies.
@Vanessa-Kris Vanessa-Kris requested a review from PromiseFru April 2, 2024 09:52
Copy link
Member

@PromiseFru PromiseFru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @Vanessa-Kris, please see my comments.

src/App.js Outdated Show resolved Hide resolved
src/Components/CountrySearch.js Outdated Show resolved Hide resolved
src/Components/DateSearch.js Outdated Show resolved Hide resolved
@PromiseFru PromiseFru changed the title Updates [Feature] Implement Data Fetching from API Apr 3, 2024
Copy link
Member

@PromiseFru PromiseFru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you @Vanessa-Kris!

@PromiseFru PromiseFru merged commit 46fd426 into staging Apr 3, 2024
2 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.

2 participants