-
Notifications
You must be signed in to change notification settings - Fork 24
Update HTTP to HTTPS & link fix #1113
base: master
Are you sure you want to change the base?
Conversation
Change links to HTTPS.
Problem: t3n link was gone, replaced it with the archived link. Change links to HTTPS. Exception: www.alphr.com, website uses self-signed certificate.
Change links to HTTPS.
Change links to HTTPS.
Change links to HTTPS.
Change links to HTTPS.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @missuniverse thanks for filing this PR! I just left some inline comments.
@@ -54,7 +54,7 @@ const PAST_COVERAGE = [ | |||
{ | |||
media: `t3n`, | |||
title: `Wie steht es um die Gesundheit des Internets? Auf Spurensuche beim Mozilla-Festival`, | |||
link: `http://t3n.de/news/mozilla-festival-internet-gesundheit-761360/3/`, | |||
link: `https://web.archive.org/web/20170112212535/http://t3n.de/news/mozilla-festival-internet-gesundheit-761360/3/`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @KevZawacki the link for tn3
on https://mozillafestival.org/media is not available anymore. Is it okay if we fix the link to https://web.archive.org/web/20170112212535/http://t3n.de/news/mozilla-festival-internet-gesundheit-761360/3/
or you have other suggestion?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pinging @KevZawacki again in case you missed this
Removed HTTPS issue with one of the links.
Change links to HTTPS when supported.
Found an invalid link and replaced it with the archived version.
Some links weren't changed because the websites didn't support it or had invalid SSL certificates.