search

Elijas / auto-youtube-subscription-playlist-2

Script automatically adds videos to playlists from Youtube channels and/or subscriptions (Youtube Collections alternative).
MIT License
252 stars 55 forks source link

refactor with max frequency per row #66

Closed JesperDramsch closed 4 years ago

JesperDramsch commented 4 years ago

Refactor of column names to make code more legible and easily changeable. Corrected column number for deletion days.

Add Column for max update frequency per row as suggested in #21

This PR only works with the following sheet: image

JesperDramsch commented 4 years ago

This way the update frequency for the sheet can be a high number like every hour, but each row is only updated according to column 3.

Works without values in column 3 and 4.

JesperDramsch commented 4 years ago

Okay, I changed the getRange codes to a call of the data array and gave an explanation in the comments with the variable to make it easier for new devs. Changed everything to the correct 0-index (sorry again).

Now the only "off by X" correction has to be made when writing to the sheet, which is explained in the sheet.

Fixed a bug with a variable name playlistId vs plalistID that came up from time to time.

Addressed all commens and implemented those changes.

JesperDramsch commented 4 years ago

Thanks for the thorough review!

JesperDramsch commented 4 years ago

@victorjzsun All points addressed.