Skip to content

Remove video links#8861

Merged
esethna merged 5 commits intomasterfrom
amyblais-videoremove
Apr 13, 2026
Merged

Remove video links#8861
esethna merged 5 commits intomasterfrom
amyblais-videoremove

Conversation

@amyblais
Copy link
Copy Markdown
Member

@amyblais amyblais commented Apr 8, 2026

Removed old release video links as they are no longer accessible.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 8, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 031b3489-7a43-4370-8122-c659036523fb

📥 Commits

Reviewing files that changed from the base of the PR and between dbe1e0d and 0a48bc0.

📒 Files selected for processing (1)
  • source/product-overview/mattermost-v10-changelog.md
💤 Files with no reviewable changes (1)
  • source/product-overview/mattermost-v10-changelog.md

📝 Walkthrough

Walkthrough

Removed three "See this walkthrough video" lines with associated video URLs from the Improvements/Highlights sections in releases v10.3, v10.2, and v10.0 of the Mattermost v10 changelog documentation. No functional changes to surrounding content.

Changes

Cohort / File(s) Summary
Documentation
source/product-overview/mattermost-v10-changelog.md
Removed six lines containing video walkthrough references (two lines per release for v10.3, v10.2, and v10.0) from changelog sections.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Remove video links' directly and clearly describes the main change: removing video links from the changelog documentation.
Description check ✅ Passed The description accurately explains that old release video links were removed because they are no longer accessible, which aligns with the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch amyblais-videoremove

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@amyblais amyblais added the 2: Editor Review Requires review by an editor label Apr 8, 2026
@amyblais amyblais requested review from Combs7th and esethna April 8, 2026 09:29
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

Newest code from mattermost has been published to preview environment for Git SHA ab08438

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

Newest code from mattermost has been published to preview environment for Git SHA 876f799

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

Newest code from mattermost has been published to preview environment for Git SHA 0a48bc0

@github-actions
Copy link
Copy Markdown
Contributor

Newest code from mattermost has been published to preview environment for Git SHA 44684f7

@esethna esethna removed the request for review from Combs7th April 13, 2026 22:24
@esethna esethna merged commit d2faaf6 into master Apr 13, 2026
4 of 5 checks passed
@esethna esethna deleted the amyblais-videoremove branch April 13, 2026 22:24
@github-actions
Copy link
Copy Markdown
Contributor

Newest code from mattermost has been published to preview environment for Git SHA f621743

@amyblais amyblais removed the 2: Editor Review Requires review by an editor label Apr 14, 2026
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.

2 participants