Fix About section QA rejections: broken links, typos, exposed internal notes#862
Closed
Copilot wants to merge 3 commits into
Closed
Fix About section QA rejections: broken links, typos, exposed internal notes#862Copilot wants to merge 3 commits into
Copilot wants to merge 3 commits into
Conversation
Collaborator
|
rebase this to docs-v2 NOT MAIN @copilot |
Agent-Logs-Url: https://github.com/livepeer/docs/sessions/78b2a345-95f0-440f-b661-0e03e4e95bad Co-authored-by: DeveloperAlly <12529822+DeveloperAlly@users.noreply.github.com>
…placeholder text Agent-Logs-Url: https://github.com/livepeer/docs/sessions/78b2a345-95f0-440f-b661-0e03e4e95bad Co-authored-by: DeveloperAlly <12529822+DeveloperAlly@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix QA review issues in About section
Fix About section QA rejections: broken links, typos, exposed internal notes
Apr 7, 2026
Collaborator
|
THIS MUST BE ON DOCS-V2 BRANCH |
Collaborator
|
@copilot is this PR Ready ? |
Collaborator
|
@copilot wrong branch. closing |
Copilot stopped work on behalf of
DeveloperAlly due to an error
April 7, 2026 11:40
Contributor
There was a problem hiding this comment.
Pull request overview
This PR addresses QA rejections in the v2 About section by fixing broken internal links, correcting a typo, and removing internal/editorial placeholder content that was visible to end users. It also adds several new LFS-tracked media assets.
Changes:
- Fixed About portal and overview navigation links (including adding missing
/v2/prefixes) and corrected “incenticises” → “incentivises”. - Removed internal notes/placeholder callouts from multiple About pages (notes, TODOs, editorial
<Danger>blocks). - Added multiple new media assets (GIF + MP4s) tracked via Git LFS.
Reviewed changes
Copilot reviewed 6 out of 15 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| v2/about/resources/livepeer-glossary.mdx | Removes duplicate front matter / internal notes / malformed heading block at top of glossary. |
| v2/about/portal.mdx | Fixes broken card hrefs in the About portal grid. |
| v2/about/mental-model.mdx | Removes an internal placeholder <Note>. |
| v2/about/livepeer-protocol/livepeer-token.mdx | Removes TODO/editorial markers and an internal <Danger> callout. |
| v2/about/livepeer-overview.mdx | Fixes typo and updates several internal links to include /v2/ prefix. |
| v2/about/core-concepts.mdx | Removes internal diagram placeholder notes. |
| v1/images/ai/cool-cat-hat-moving.gif | Adds a GIF via LFS (currently appears unused in-repo). |
| snippets/assets/media/videos/streamplace.mp4 | Adds a video via LFS (currently appears unused in-repo). |
| snippets/assets/media/videos/nytvlivepromo.mp4 | Adds a video via LFS (currently appears unused in-repo). |
| snippets/assets/media/videos/nytv.live.mp4 | Adds a video via LFS (currently appears unused in-repo). |
| snippets/assets/media/videos/LivepeerStudio.mp4 | Adds a video via LFS (currently appears unused in-repo). |
| snippets/assets/media/videos/livepeer-founders-post.mp4 | Adds a video via LFS (referenced elsewhere in v2 home/component-library docs). |
| snippets/assets/media/videos/HeroBackground.mp4 | Adds a large hero background video via LFS (referenced elsewhere in component-library docs). |
| snippets/assets/media/videos/frameworks.mp4 | Adds a video via LFS (currently appears unused in-repo). |
| snippets/assets/media/videos/daydream.mp4 | Adds a video via LFS (referenced by TitledVideo examples/docs). |
| **Roles**: Stake LPT, earn rewards, participate in governance and voting | ||
| </Card> | ||
| <Card title="Platforms? Workers?" icon="laptop-code" cta="Go to Platforms" href="/developers/portal"> | ||
| <Card title="Platforms? Workers?" icon="laptop-code" cta="Go to Platforms" href="/v2/developers/portal"> |
There was a problem hiding this comment.
The card title "Platforms? Workers?" reads like an unresolved placeholder and will be user-visible in the About overview. Please replace it with a definitive label (and, if needed, adjust the description/CTA to match).
Suggested change
| <Card title="Platforms? Workers?" icon="laptop-code" cta="Go to Platforms" href="/v2/developers/portal"> | |
| <Card title="Platforms" icon="laptop-code" cta="Go to Platforms" href="/v2/developers/portal"> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
docs-v2(reset hard toorigin/docs-v2)## ====heading leak, orphaned brainstorm section