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

Refactor video_metadata to use xb #254

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

deldesir
Copy link
Collaborator

@deldesir deldesir commented Sep 12, 2024

This PR refactors the video_metadata function to use SQLAlchemy (via xb.py) instead of raw SQLite. This aims to improve its maintainability and scalability by leveraging SQLAlchemy's ORM capabilities.

  • Tested on Ubuntu 24.04

@deldesir deldesir added the enhancement New feature or request label Sep 12, 2024
@deldesir deldesir requested a review from holta September 12, 2024 15:59
@deldesir deldesir self-assigned this Sep 12, 2024
@holta
Copy link
Member

holta commented Nov 17, 2024

FYI above commit f2a2570 should be clearly explained + contextualized here:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants