Fix broken video embed and link wording in framework capability docs#2195
Open
flanakin wants to merge 4 commits into
Open
Fix broken video embed and link wording in framework capability docs#2195flanakin wants to merge 4 commits into
flanakin wants to merge 4 commits into
Conversation
- understand/reporting.md: the FinOps Foundation video embed id was corrupted (CVTJLdcozj1eEpxT, 16 chars; valid YouTube ids are 11), so the video didn't load. Replaced with pD9NeBOvspU (the Reporting & Analytics video for playlist PLUSCToibAswlDSQdehKhi7ysP2hmetigl, resolved via YouTube oEmbed). - quantify/budgeting.md: "see to the [Budgeting]" -> "see the [Budgeting capability]" and added the trailing slash on the capability URL, matching the wording and link format used by the sibling capability pages.
microsoft-github-policy-service
Bot
added
the
Needs: Review 👀
microsoft-github-policy-service
Bot
added
the
Tool: FinOps guide
Contributor
There was a problem hiding this comment.
Pull request overview
This PR applies two small documentation fixes in the FinOps Framework capability articles under docs-mslearn/, correcting a broken YouTube video embed and improving wording/link consistency for a “Learn more” reference.
Changes:
- Fix the Reporting & analytics page’s FinOps Foundation video embed by replacing an invalid YouTube video ID.
- Improve Budgeting page wording (“see the Budgeting capability”) and normalize the FinOps Framework capability URL to include a trailing
/.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| docs-mslearn/framework/understand/reporting.md | Updates the embedded YouTube video ID for the FinOps Foundation video playlist reference. |
| docs-mslearn/framework/quantify/budgeting.md | Adjusts “Learn more” sentence wording and normalizes the FinOps Framework capability link format. |
RolandKrummenacher
approved these changes
Jun 29, 2026
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.
🛠️ Description
Two small doc fixes in the FinOps Framework capability articles, found while reviewing the published pages:
understand/reporting.md— the FinOps Foundation video embed id was corrupted:CVTJLdcozj1eEpxTis 16 characters, but valid YouTube ids are 11, so the video did not load. Replaced withpD9NeBOvspU, keeping the?list=playlist param.PLUSCToibAswlDSQdehKhi7ysP2hmetigl), which returns title "Reporting & Analytics" / author "FinOps Foundation". It's a confirmed live FinOps Foundation video for that playlist; if a maintainer knows the canonical Reporting & Analytics video differs, please adjust the id — the playlist param is unchanged either way.quantify/budgeting.md— "see to the [Budgeting]" → "see the [Budgeting capability]", and added the trailing slash on the capability URL, to match the wording and link format used by every sibling capability page (e.g. reporting, allocation, unit-economics).📋 Checklist
🔬 How did you test this change?
Docs-only change — no code, deployment, or schema impact.