Skip to content

Remove stream visualizer glitch demo links #40186

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 3, 2025
Merged

Remove stream visualizer glitch demo links #40186

merged 1 commit into from
Jul 3, 2025

Conversation

chrisdavidmills
Copy link
Contributor

Description

The MDN Streams API documentation has several pages that link to this demo hosted on Glitch.

Because Glitch is going away, I've decided to remove these links. I don't see this demo as being vital to the documentation, plus it is fairly complicated, and I don't think we should take on the role of maintaining it.

If it gets moved somewhere else, we can always link to the new location in the future.

Motivation

Additional details

Related issues and pull requests

@chrisdavidmills chrisdavidmills requested a review from a team as a code owner July 3, 2025 10:59
@chrisdavidmills chrisdavidmills requested review from sideshowbarker and removed request for a team July 3, 2025 10:59
@github-actions github-actions bot added Content:WebAPI Web API docs size/s [PR only] 6-50 LoC changed labels Jul 3, 2025
@chrisdavidmills chrisdavidmills requested review from bsmth and a team July 3, 2025 11:01
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jul 3, 2025

Preview URLs (7 pages)
External URLs (14)

URL: /en-US/docs/Web/API/ReadableByteStreamController
Title: ReadableByteStreamController

URL: /en-US/docs/Web/API/ReadableStream
Title: ReadableStream

URL: /en-US/docs/Web/API/ReadableStreamBYOBReader
Title: ReadableStreamBYOBReader

URL: /en-US/docs/Web/API/ReadableStreamDefaultController
Title: ReadableStreamDefaultController

URL: /en-US/docs/Web/API/ReadableStreamDefaultReader
Title: ReadableStreamDefaultReader

URL: /en-US/docs/Web/API/TransformStream
Title: TransformStream

URL: /en-US/docs/Web/API/WritableStream
Title: WritableStream

bsmth
bsmth approved these changes Jul 3, 2025
View reviewed changes
Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

Thank you!

@bsmth bsmth removed the request for review from sideshowbarker July 3, 2025 11:07
@bsmth bsmth merged commit 7d37e07 into mdn:main Jul 3, 2025
7 checks passed
@chrisdavidmills chrisdavidmills deleted the remove-stream-visualizer-demo-links branch July 3, 2025 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bsmth bsmth bsmth approved these changes

Assignees
No one assigned
Labels
Content:WebAPI Web API docs size/s [PR only] 6-50 LoC changed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chrisdavidmills @bsmth