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

Print videos added to the database. #114

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

buzzingwires
Copy link

@buzzingwires buzzingwires commented Apr 24, 2023

What

When adding videos to a playlist, print the ID of the added video, as well as the title and name of playlist itself.

Why

To accomodate the use case of invoking ytcc update to check for new videos, then ytcc play shortly afterwards to play one of the new videos, without needing to invoke ytcc list to view the ID.

How

Print the title, playlist, and ID of the video when adding it to the database. The previous processing message is changed to the debug log level

The ID of the video is included in the new message, if the database
supports lastrowid.

The video processing message is now on the debug level.
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.

1 participant