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

remove use of mdn.mozillademos.org #6204

Merged
merged 1 commit into from
Jun 22, 2021

Conversation

peterbe
Copy link
Contributor

@peterbe peterbe commented Jun 21, 2021

What was wrong/why is this fix needed? (quick summary only)

Lots of samples use the deprecated https://mdn.mozillademos.org/files/... URLs.

Issue number (if there is an associated issue)

n/a

Anything else that could help us review it

See existing ongoing work: https://github.com/mdn/content/pulls?q=is%3Apr+is%3Aclosed+mdn.mozillademos.org

@peterbe peterbe requested a review from a team as a code owner June 21, 2021 20:04
@peterbe peterbe requested review from Rumyra and removed request for a team June 21, 2021 20:04
@peterbe
Copy link
Contributor Author

peterbe commented Jun 21, 2021

This is actually the last IMAGE that I can find that uses mdn.mozillademos.org.
There are other live samples that still use mdn.mozillademos.org but not for images. In particular, I think there are a couple of .html and .ttf (fonts) files that still use it.

@github-actions
Copy link
Contributor

Preview URLs

Flaws

None! 🎉

External URLs

URL: /en-US/docs/Games/Techniques/Crisp_pixel_art_look
Title: Crisp pixel art look with image-rendering
on GitHub

No new external URLs

@sideshowbarker sideshowbarker merged commit 7ae5b3b into mdn:main Jun 22, 2021
Gregoor pushed a commit that referenced this pull request Jun 22, 2021
@peterbe peterbe deleted the remove-use-of-mdnmozillademosorg branch June 22, 2021 19:26
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants