Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix outdated / broken links #958

Merged
merged 2 commits into from
Sep 21, 2021
Merged

Fix outdated / broken links #958

merged 2 commits into from
Sep 21, 2021

Conversation

mashb1t
Copy link
Collaborator

@mashb1t mashb1t commented Sep 14, 2021

Fixes outdated / broken links

see test results in #957
@johackim fyi

see test results of #957
Copy link
Collaborator

@agebhar1 agebhar1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work @mashb1t 👍

I'm fine with removing the monitoring for YouTube (Playlists).

@mashb1t mashb1t merged commit 48e1a49 into master Sep 21, 2021
@mashb1t mashb1t deleted the fix-links branch September 21, 2021 18:27
@mashb1t
Copy link
Collaborator Author

mashb1t commented Sep 21, 2021

@agebhar1 as YouTube links are sometimes redirected and sometimes not, we should keep them to at least sometimes get notified that a video / playlist went offline or isn't available anymore

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants