Skip to content

Rewrite Mux Integration#429

Merged
pinkeshmars merged 4 commits intomainfrom
rewrite/mux-broadcast
Mar 10, 2026
Merged

Rewrite Mux Integration#429
pinkeshmars merged 4 commits intomainfrom
rewrite/mux-broadcast

Conversation

@pinkeshmars
Copy link
Collaborator

Description

Rewrite Mux Integration

Linear ticket and magic word Fixes DEVR-486

Type of change

  • Typo fix
  • New feature
  • Enhancement to current docs
  • Removed outdated references
  • Update assets

@pinkeshmars pinkeshmars requested a review from PoojaB26 July 18, 2025 11:44
@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR rewrites the Mux livestream integration documentation by consolidating the previous multi-page Mux docs into a single /integrations/mux page, removing the old Mux doc pages/assets, and adding redirects so existing links continue to resolve.

Changes:

  • Replace the old Mux “initial setup” + “getting started” docs with a single consolidated Mux Livestream doc at /integrations/mux.
  • Remove outdated Mux doc pages/category and associated legacy image assets.
  • Add Firebase Hosting redirects for the retired Mux routes and update an internal “Also see” link to the new doc.

Reviewed changes

Copilot reviewed 6 out of 28 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
firebase.json Adds redirects from deprecated Mux routes to the new consolidated /integrations/mux page.
docs/ff-integrations/streaming/mux/integrate-mux.md Removes the legacy Mux “getting started” page (superseded by the new consolidated doc).
docs/ff-integrations/streaming/mux/initial-setup.md Removes the legacy Mux “initial setup” page (superseded by the new consolidated doc).
docs/ff-integrations/streaming/mux/category.json Removes the legacy Mux sidebar category metadata along with the retired docs.
docs/ff-integrations/streaming/integrate-mux.md Adds the new consolidated Mux Livestream documentation at slug /integrations/mux.
docs/ff-integrations/streaming/imgs/get-past-stream-id.avif Adds a new asset used by the updated “past livestream” playback flow section.
docs/ff-concepts/file-handling/file-handling.md Updates the cross-link to point to the new consolidated Mux doc.
docs/ff-integrations/streaming/mux/img.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_1.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_2.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_3.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_4.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_5.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_6.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_7.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_8.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_9.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_10.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_11.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_12.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_13.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_14.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_15.png Removes legacy image asset associated with deleted Mux docs.
docs/ff-integrations/streaming/mux/img_16.png Removes legacy image asset associated with deleted Mux docs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +152 to +153
- [**GET /video/v1/live-streams/`{LIVE_STREAM_ID}`**](https://www.mux.com/docs/api-reference/video/live-streams/get-live-stream): Retrieves the livestream details including `recent_asset_ids` array. Extract the Asset ID from this api response.
- [**GET /video/v1/assets/`{ASSET_ID}`**](https://www.mux.com/docs/api-reference/video/assets/get-asset): Fetches the asset details to get its playback ID from the `playback_ids` array.
Copy link

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

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

In the Mux API list, “api response” should be capitalized as “API response” (and ideally consistently use “API” in this section) to match standard terminology and other docs.

Copilot uses AI. Check for mistakes.
Comment on lines +2416 to +2424
{
"source": "/integrations/mux/initial-setup",
"destination": "/integrations/mux",
"type": 301
},
{
"source": "/integrations/mux/getting-started",
"destination": "/integrations/mux",
"type": 301
Copy link

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

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

These new redirects are good, but they introduce redirect chains because several existing redirects still point at the removed pages (e.g. /settings-and-integrations/integrations/mux-livestream/integrations/mux/initial-setup, and MuxBroadcast widget redirects → /integrations/mux/getting-started). Consider updating those existing redirects to point directly to /integrations/mux to avoid extra 301 hops and keep the redirect map simpler.

Copilot uses AI. Check for mistakes.
@pinkeshmars pinkeshmars merged commit 7e11e44 into main Mar 10, 2026
1 check passed
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.

3 participants