unople / youtubeplaylistmanager Goto Github PK

Next to selected playlist (or Uncategorized etc) a refresh button that refreshes the selected thing in the combobox

Refresh above just the playlist part to refresh only the playlists and no items inside

Refresh above selected video to refresh just that one

Refresh options when clicking the big refresh button too?

• Refresh all
• Refresh only current playlists
• etc?

I have two programming playlists... for some reason. They both appear on the list. I can't select one over the other in the combobox.

For some videos as well, it looks like the videos aren't in either programming playlist.

In progress:
Done:

• Add playlist
• Remove playlist
• Add a video to the playlist we're looking at
• Remove a video from the playlist we're looking at

Refresh.

Then close the window and refresh again.

Eventually, the window is cleared properly, and no more logs get appended.

• Change the update window to more of a logs window (can show or hide it, and it doesn't clear logs over the normal clamp to 10K)
• Make all of the logs go through the logs window
• Make another way of cancelling active tasks (like the refresh from YouTube)
• Show proper progress bar for things

• Work out and store an average of how long running methods (like GetPlaylistData) takes
• Use that average to set the step counts for each thing we want to do in a run
• Exclude runs when calculating averages that did things like reauth, or runs that were cancelled

Logging added in commit f80b0b1

Need to be able to do the following:

1. Store the API key somewhere
2. It needs to be secure

At the moment it's stored using AddUserSecrets from microsoft user secrets

With the change to async, basically everything is running in different threads now.

Need a better, more reliable way of ensuring multiple calls to the DB context are done in the same thread, especially with the DB context being thread static. This needs to be an architectural / code design change in the repository layer

• Fix issue so videos get saved properly - done in commit 87e4dff
• Research out how to properly thread out calls to the database in a multithreaded / async context. I think part of the reason why I made the DB layer ThreadStatic was because there was more than one thread accessing the DB Context, and queueing up changes - which errored.
• Make changes to the DI layer / the provider to follow whatever the research says

Repro steps:
Type in a, then type in SDF before it tells you how many results there were from the search
You'll get more results than you should - it has cleared the list of videos, but kept populating it from the old search

To help with new developers coming into this application

It's meant to walk people through the code at an extremely high level

https://news.ycombinator.com/item?id=26048784
https://matklad.github.io/2021/02/06/ARCHITECTURE.md.html

Probably becasue the DI hasn't initialized the google stuff yet. Maybe it's a good idea to call something on a background thread to initialize it?

At the moment I'm only getting the smallest size, but I want to get larger size so the thumbnail displayed on the right isn't so blurry

Things that make this tricky:

1. Need to store more than one thumbnail size, but don't want to have to load all of them all the time
2. Need to populate the larger sized thumbnails for every video
3. Only load the large sized thumbnails when selecting a video - but how to make that performant?
4. Allow for larger sized thumbs not existing, fall back to tiny version

It's almost worth having some kind of 'update new data' method / abstraction for videos. Needs some thought though

todo:

• Get thumbnail data for new videos from YouTube - done in 1d2b561
• Get thumbnail data for old videos
• Get actual thumbnail data
• Load the larger thumbnail instead of the tiny version, but fall back to old version

Steps to reproduce:

1. Open UI
2. Select playlist via combobox
3. Press refresh button - you see logs in the UI
4. Close the update UI
5. Press refresh button again - no logs in the UI at all