This pull request refactors the CSV import functionality for the queue API by introducing a dedicated csv module, adds a new (template) Apollo CSV import endpoint, and cleans up test files. #91
Merged
goldlabelapps merged 3 commits intomasterfrom Apr 18, 2026
Conversation
Rename app/api/queue/routes/import_linkedin.py to app/api/queue/import/linkedin.py to reorganize the queue import handlers. No functional changes were made (100% similarity/only a file move).
Reorganize import-related endpoints into a dedicated import_ package. Added package markers (__init__.py) and a placeholder README, moved the existing linkedin importer into app/api/queue/import_ and updated the queue router import paths accordingly. Also added a new apollo.py stub endpoint (/queue/import/apollo) that provides a template for CSV import (includes TODOs for CSV parsing and DB insertion). No functional import logic implemented yet; this change is primarily structural to group import handlers.
Refactor CSV import code by renaming the import_ package to csv and moving apollo/linkedin import modules under app/api/queue/csv. Update FastAPI route paths from /queue/import/... to /queue/csv/... and register the apollo CSV import router in app/api/queue/__init__.py. Add a placeholder CSV README and remove the old import_ package files and an obsolete test (tests/test_prompt.py). This reorganizes import endpoints for clearer CSV-related grouping.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The main themes are improved modularity for CSV imports, standardization of import routes, and removal of obsolete or redundant tests.
CSV Import Refactoring and Additions:
app/api/queue/csvpackage with__init__.pyand a placeholderREADME.mdto organize CSV import routes. [1] [2]apollo.pyfile providing a template endpoint for importing Apollo CSV data at/queue/csv/apollo. The implementation is currently a stub with placeholder logic for future development.routes/import_linkedin.pytocsv/linkedin.py, and updated the endpoint to/queue/csv/linkedinfor consistency.API Router Updates:
app/api/queue/__init__.pyto register the new CSV import routers (linkedinandapollo), removing the old LinkedIn import router and including the new ones under thecsvnamespace. [1] [2]Test Cleanup:
tests/test_prompt.pyfile, which contained both real and mocked LLM API tests, likely to reduce redundancy or because the tests are no longer relevant.