on4r / youtube-playlist-tracker Goto Github PK

View Code? Open in Web Editor NEW

0.0 0.0 0.0 778 KB

Take snapshots of playlists

JavaScript 78.48% CSS 5.16% Shell 3.59% EJS 12.77%

youtube-playlist-tracker's People

Contributors

Watchers

youtube-playlist-tracker's Issues

Throw error and exist application if there is no such database file

https://flaviocopes.com/node-terminate-program/

youtube-playlist-tracker/modules/database.js

Lines 7 to 10 in 13b36c2

    
           function init(pathToDatabase) { 
        
           	dbLocation = pathToDatabase 
        
           	console.log("DATABASE: using", pathToDatabase) 
        
           }

Add option to export all video URLs from a playlist

Primary add solo as a CLI function.

SELECT video_id FROM playlists_videos WHERE playlist_id=
SELECT url FROM videos WHERE id=

Create a .txt file where each row contains one url.

Add favicon

Lupen oder Disketten Emoji

Move update youtube-dl to own module

youtube-playlist-tracker/modules/cron.js

Line 42 in e9977d1

async function updateYoutubeDl() {

Improve logs

Add info about added videos to console.log if there are less the 5 videos added.

Add padding and primary color to apple-touch-icon

primar-color: orangered;

Replace inline callbacks with arrow functions

Add playlists-overview page

admin mode - on!

Generalize naming

filenames, functions, variables, etc. should be the same across the board. thx.

Add email notifications

Receive an email whenever a video of a playlist you watch gets deleted.

Rename to controller.js

modules/updateDatabaseLogic.js

and include InProgress.

Fix margin-top

Add validation to 'update-video' route

Validate user_title

youtube-playlist-tracker/modules/router.js

Lines 69 to 88 in ae036c1

    
           Router.post("/videos/:id/update", async function(req, res) { 
        
           	const video_id = req.params.id 
        
           	const user_title = req.body.user_title 
        
           	// validate user_title (serialize, escape, w/e) 
        
           	try { 
        
           		await Database.open() 
        
           		await Database.updateVideo(video_id, {user_title}) 
        
           	} catch (error) { 
        
           		console.log("Error while POST /videos/:id/update/", error) 
        
           	} finally { 
        
           		await Database.close() 
        
           	} 
        
           	res.redirect(`${stripQueryParams(req.headers.referer)}?updated=${video_id}`) 
        
           	return 
        
           })

Integrate new update behavior for youtube-dl

Use an aproach like youtube-dl itself to update the binary via LATEST_VERSION see here.

Nest inputs into labels

or add id to input.

Fix mass delete and create

Passiert manchmal und ohne offensichtlichen Grund:

In einem update:

Deleted 474 Joint-Relations of playlist 4

und direkt danach:

Created 474 Joint-Relations

WHY?!

Potential error if youtube-dl is outdated

Attention:

youtube-playlist-tracker/modules/router.js

Lines 48 to 49 in e5cd5c5

    
           // start the async "youtube-dl and fill database" script 
        
           await Controller.parsePlaylistAndUpdateTables(playlist)

Here we call the update playlist script without checking if youtube-dl is up-to-date. This could lead to an error.
The same thing applys if you call the npm scripts to update playlist(s).

Currently we are only checking for updates in the cron module.

Fix page title

page title depends on current route and should not be set here.

youtube-playlist-tracker/modules/router.js

Line 180 in e9977d1

    
           res.locals.page_title = `${playlist.title} by ${playlist.uploader_id} - Research Mode`

YouTube sometimes also removes a deleted video from your playlist

Nur zur Info. Deshalb sollte man überlegen ob die joint-relations wirklich gelöscht werden sollten.
Anderseits kann es auch nice sein wenn uralter müll automatisch wegfällt. Hmm. Mal schaun.
Affects:

youtube-playlist-tracker/modules/controller.js

Lines 163 to 167 in ae036c1

    
           // check if video got removed from playlist (by user) 
        
           currentVideoUrls.forEach(url => { 
        
           	if (!latestVideoUrls.includes(url)) 
        
           		removedVideoUrls.push(url) 
        
           })

Add cli/addPlaylist.js npm script

first close #12

Add margin to body (again)

Add margin to the sides

youtube-playlist-tracker/public/styles.css

Line 7 in d3ed60d

margin: 3rem 0 0 0;

Damit die Elemente on mobile einen Abstand zum Rand haben

	function init(pathToDatabase) {
	dbLocation = pathToDatabase
	console.log("DATABASE: using", pathToDatabase)
	}

	Router.post("/videos/:id/update", async function(req, res) {

	const video_id = req.params.id
	const user_title = req.body.user_title

	// validate user_title (serialize, escape, w/e)

	try {
	await Database.open()
	await Database.updateVideo(video_id, {user_title})
	} catch (error) {
	console.log("Error while POST /videos/:id/update/", error)
	} finally {
	await Database.close()
	}

	res.redirect(`${stripQueryParams(req.headers.referer)}?updated=${video_id}`)
	return

	})

	// start the async "youtube-dl and fill database" script
	await Controller.parsePlaylistAndUpdateTables(playlist)

	// check if video got removed from playlist (by user)
	currentVideoUrls.forEach(url => {
	if (!latestVideoUrls.includes(url))
	removedVideoUrls.push(url)
	})

on4r / youtube-playlist-tracker Goto Github PK

youtube-playlist-tracker's People

Contributors

Watchers

youtube-playlist-tracker's Issues

Recommend Projects

Recommend Topics

Recommend Org