Conversation
|
|
# Conflicts: # firebase.json
There was a problem hiding this comment.
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.
| - [**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. |
There was a problem hiding this comment.
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.
| { | ||
| "source": "/integrations/mux/initial-setup", | ||
| "destination": "/integrations/mux", | ||
| "type": 301 | ||
| }, | ||
| { | ||
| "source": "/integrations/mux/getting-started", | ||
| "destination": "/integrations/mux", | ||
| "type": 301 |
There was a problem hiding this comment.
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.
Description
Rewrite Mux Integration
Linear ticket and magic word Fixes DEVR-486
Type of change