data : add new API ELT pipeline data engineering project - #148
Conversation
|
@Harshan07-web is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel. A member of the Team first needs to authorize it. |
komalharshita
left a comment
There was a problem hiding this comment.
Good contribution overall — this is a meaningful addition to the DevPath dataset and introduces a stronger intermediate-level data engineering project compared to typical beginner project submissions.
What’s done well:
- the project idea is relevant and industry-aligned,
- the roadmap progression is structured logically,
- the learning goals are clearly explained,
- and the PR scope is clean and focused.
A few small improvements are recommended before merge:
- Fix minor wording/typo issues:
- “Upload a API url” → “Upload an API URL”
- “reponse” → “response”
- “datatable” → “dataset” or “data table”
-
os moduledoes not really fit well as a project skill/tech stack item. Consider replacing or removing it. -
The starter code skeleton would be stronger if basic function stubs were included (for example
fetch_api_data(),clean_data(), etc.) instead of comments only.
Overall, this is a solid intermediate-level project addition and a valuable contribution to the repository.
|
@komalharshita Thank you for being specific about the improvements that were required, I have taken care of : |
|
@Harshan07-web kindly resolve the merge conflicts |
|
@komalharshita I have resolved the merge conflict. |
Summary
Added a new data engineering project which fetches data from public api, then normalizes the JSON data, converts them to a pandas dataframe, handles missing and duplicate values and the exports the CSV file. This CSV file can further be used in data analytics projects.
Related Issue
Closes #147
Type of Change [required]
data/projects.jsonWhat Was Changed
data/projects.jsonid 9API ETL Pipelinestartercode/api_data_pipeline.pyHow to Test This PR
git checkout your-branch-namepip install -r requirements.txtpython app.pypython tests/test_basic.pyExpected test output:
Test Results
Screenshots (if UI change)
Self-Review Checklist [required]
feat/,fix/,docs/,data/,style/,test/python tests/test_basic.pyand all 27 tests passflake8 .locally and there are no errorsprint()orconsole.log()debug statementsNotes for Reviewer
Please note that the sample code for this project is not fully written. Only the skeleton of the sample code is created(comments), after the project is approved, I'll be happy to work on the complete sample code.