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

issue #74: Implement batch import route in Backend #75

Merged
merged 15 commits into from
Jun 6, 2024

Conversation

MaxenceGui
Copy link

@MaxenceGui MaxenceGui commented Apr 12, 2024

  • Create the documentation
  • Create the bacth import tests
  • Create the batch import route
  • Create the connexe route to send the frontend the data to collect the right metadata

@MaxenceGui MaxenceGui linked an issue Apr 12, 2024 that may be closed by this pull request
3 tasks
@MaxenceGui MaxenceGui changed the title issue #74: Create test and documentation file issue #74: Implement batch import route in Backend Apr 12, 2024
@MaxenceGui MaxenceGui force-pushed the 74-implement-api-route-for-picture-batch-import branch from 4e8e0eb to 3bc4fd0 Compare April 22, 2024 17:44
@MaxenceGui MaxenceGui added the On Hold Issue that are on hold label Apr 25, 2024
@MaxenceGui
Copy link
Author

MaxenceGui commented Apr 25, 2024

Pull Request Status Update

Summary

This pull request was open to implement the batch upload functionality in the backend. The PR is On Hold because the work is important, but no developer is currently on board to work on it (Apr 25).

When on board, the developer should know:

  • Documentation on the functionality is located at /docs/nachet-batch-import-documentation.md
  • The azure_api_storage folder was renamed storage to reflect soon to be change with the incorporation of nachet-datastore
  • datastore_storage_api.py is not completed and will need rework (or can be eliminated if the dev doesn't see a utility for it)
  • The batch import is divided in two route:
  • The batch import functionality is currently divided in three function:
  • However, these function can be rework and remove since the functionality has been tests or proven to work
  • Tests for the batch import are located at /tests/test_image_batch_import.py

Notes

This is still a work in progress and constant communication with the frontend dev and the database dev will help the backend dev achieve is objective.

This PR and issue are part of an epic:

The pull request for setting the pyproject.toml to import nachet-datastore:

@MaxenceGui MaxenceGui force-pushed the 74-implement-api-route-for-picture-batch-import branch from 9affecc to 3a2ba7f Compare April 25, 2024 19:19
@sylvanie85 sylvanie85 merged commit b7b8de1 into main Jun 6, 2024
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
On Hold Issue that are on hold
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement API route for picture batch import
2 participants