Update "Last Update (Timestamp)" Column Dynamically When Quota is Reached#127
Update "Last Update (Timestamp)" Column Dynamically When Quota is Reached#127davisjk wants to merge 24 commits into
Conversation
Currently does nothing but gets read and validated
…pefully Trying to debug "Exception: The number of rows in the range must be at least 1."
davisjk
changed the title
|
not anymore |
davisjk
changed the title
…ill need to update adding videos to playlists and handling exceeding quota there
|
|
||
| 2. Setup your playlists and channels (white cells in the Sheet): | ||
| 1. Setup your playlists and channels (white cells in the Sheet): | ||
| - Clear all existing white rows, they are just examples (i.e. delete the 3 rows). |
There was a problem hiding this comment.
Might need a larger indent here, seems like it's resetting the ordered list count
There was a problem hiding this comment.
Should be fixed now. Looks good there and I also noticed a couple other places it was misaligned
| if (nextTime && dateDiff <= nextTime) { | ||
| var lastDate = new Date(lastTimestamp); | ||
| var lastTime = lastDate.getTime(); | ||
| var dateDiff = Date.now() - lastTime; |
There was a problem hiding this comment.
Instead of this, we can just subtract two date objects, aka new Date() - lastDate, and we don't need to create lastTime
|
What if instead of catching the quota error everywhere, we pushed it to the top level. Since quota refreshes daily, there should be no need to continue the script if a single quota error is hit. |
|
If you can also show how you've tested these changes, it'll help me review much faster. Otherwise I'll need to test it myself. |
Sorry I've had a lot going on but I'm back and I'll try to address your comments soon |
2 participants
No description provided.