mattwright324 / youtube-comment-suite

Download YouTube comments from numerous videos, playlists, and channels for archiving, general search, and showing activity.
MIT License
275 stars 46 forks source link

Last refresh time per group? #93

Open bernard01 opened 3 years ago

bernard01 commented 3 years ago

I know it might be a bit tricky to do this. And I don't really know how it is supposed to work at present. So I am just writing what I thought would be ideal, based on my use case as follows:

I use a large group with hundreds of videos that I can afford to refresh only every two hours because of quota and volume.

Then I use a small group that contains only a single video that I am actively watching. This one I want to refresh every 15 minutes or so.

I would expect that the time since last refresh is stored per group so I know when the 2 hours of the large group are over, when I switch from the small group to the large group. I think at present, the system does not do that, because in the large group I see the time since last refresh of the small group. I am assuming that the system cannot know whether any videos in the large group need refreshing from refreshing only the small group. I might be wrong there.

Thanks for making YCS.

mattwright324 commented 3 years ago

How it works currently probably isn't the best way to do it though I am storing the last time a group item was refreshed and am using that to determine when a group was last refreshed by the most recent date on the group items in its list. So if you have two groups and they share an item (channel, playlist, video) if you refresh one it will change the date on the other too.

https://github.com/mattwright324/youtube-comment-suite/blob/f45204bb40d45edca27f6ec2d1d0e0e53658e808/src/main/resources/io/mattw/youtube/commentsuite/db/sql/dql_get_group_last_checked.sql#L1-L3

It could make more sense to store a last_refreshed column on the groups table and/or display the last checked on each of the items in the group item list.