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

Implement video data retrieval & saving #35

Merged
merged 27 commits into from
Nov 3, 2024
Merged

Implement video data retrieval & saving #35

merged 27 commits into from
Nov 3, 2024

Conversation

leingenm
Copy link
Owner

@leingenm leingenm commented Oct 26, 2024

Pull Request

Description

Introduced controller and service layer that handles data retrieval and saving into the database.

Videos can be imported by providing a CSV file or a list of video IDs.

Video data retrieval happens in batches and is handled using transactions. If a transaction fails neither video IDs nor video data is saved.

Implemented an endpoint allowing checking of what transactions have failed and which were successful.

Related Issue

Issue #32

Checklist

Please review and check all applicable items:

  • I have tested my changes thoroughly
  • My code follows the project's coding standards and guidelines
  • I have added necessary documentation (if applicable)
  • I have updated relevant tests (if applicable)
  • I have reviewed my changes to ensure there are no unnecessary changes
  • My pull request has a descriptive title and includes a detailed description of the changes

Additional Notes

  • Updated collections in Postman
  • Disabled Liquibase in tests. Created a separate Spring profile for testing and updated GH Actions step to use it when running tests in CI

Screenshots (if applicable)

@leingenm leingenm requested a review from daverbk October 26, 2024 15:45
@leingenm leingenm self-assigned this Oct 26, 2024
@leingenm leingenm linked an issue Oct 26, 2024 that may be closed by this pull request
7 tasks
@leingenm leingenm marked this pull request as ready for review October 26, 2024 16:48
@daverbk
Copy link
Collaborator

daverbk commented Oct 27, 2024

image
wow, it's so big

@daverbk daverbk merged commit 2a33dee into main Nov 3, 2024
3 checks passed
@daverbk daverbk deleted the issue-32 branch December 25, 2024 13:13
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.

Implement video data retrieval & saving
2 participants