Skip to content

Fix: Remove broken feedback link in Optimization workbook#2038

Merged
flanakin merged 4 commits intodevfrom
flanakin/fix-workbook-feedback-link
Mar 3, 2026
Merged

Fix: Remove broken feedback link in Optimization workbook#2038
flanakin merged 4 commits intodevfrom
flanakin/fix-workbook-feedback-link

Conversation

@flanakin
Copy link
Collaborator

@flanakin flanakin commented Mar 3, 2026

🛠️ Description

Fixed the broken Office Forms feedback link in the Cost Optimization workbook that was returning a 404 error. Consolidated the two feedback links (Office Forms survey and GitHub issues) into a single GitHub issues link for a cleaner, unified feedback experience.

Changes:

  • Removed the broken aka.ms/advisor_cost_wb_feedback link (Office Forms)
  • Updated text from "Submit any issues" to "Submit feedback or issues"
  • Kept only the GitHub issues link (aka.ms/costworkbookfeedback)

Fixes #1673

📷 Screenshots

Before After
image image

📋 Checklist

🔬 How did you test this change?

  • 🤏 Lint tests
  • 🤞 PS -WhatIf / az validate
  • 👍 Manually deployed + verified
  • 💪 Unit tests
  • 🙌 Integration tests

📦 Deploy to test?

  • Hubs + ADX (managed)
  • Hubs + Fabric (manual) — URI:
  • Hubs (manual)
  • Hubs (no data)
  • Workbooks
  • Alerts

🙋‍♀️ Do any of the following that apply?

  • 🚨 This is a breaking change.
  • 🤏 The change is less than 20 lines of code.

📑 Did you update `docs/changelog.md`?

  • ✅ Updated changelog (required for `dev` PRs)
  • ➡️ Will add log in a future PR (feature branch PRs only)
  • ❎ Log not needed (small/internal change)

📖 Did you update documentation?

  • ✅ Public docs in `docs` (required for `dev`)
  • ✅ Public docs in `docs-mslearn` (required for `dev`)
  • ✅ Internal dev docs in `docs-wiki` (required for `dev`)
  • ✅ Internal dev docs in `src` (required for `dev`)
  • ➡️ Will add docs in a future PR (feature branch PRs only)
  • ❎ Docs not needed (small/internal change)

Removed the Office Forms feedback link that was returning a 404 error
and consolidated feedback to use a single GitHub issues link.

Changes:
- Removed aka.ms/advisor_cost_wb_feedback (Office Forms)
- Updated text from "Submit any issues" to "Submit feedback or issues"
- Consolidated to single GitHub feedback link (aka.ms/costworkbookfeedback)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 3, 2026 02:16
@microsoft-github-policy-service microsoft-github-policy-service bot added Micro PR 🔬 Very small PR that should be especially easy for newcomers Needs: Review 👀 PR that is ready to be reviewed labels Mar 3, 2026
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

Removes a broken feedback link from the Cost Optimization workbook content and consolidates feedback routing to the GitHub issues link, with a corresponding changelog update for the next release documentation.

Changes:

  • Removed the Office/Microsoft Forms feedback link from the Optimization workbook and kept a single GitHub feedback/issues link.
  • Updated workbook feedback wording to “Submit feedback or issues”.
  • Documented the fix in the FinOps toolkit changelog (v14 workbooks section).

Reviewed changes

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

File Description
src/workbooks/optimization/workbook.json Updates the embedded “Feedback” markdown to remove the broken Forms link and keep the GitHub link.
src/workbooks/optimization/CostOptimization.workbook Mirrors the same feedback-link consolidation in the exported workbook artifact.
docs-mslearn/toolkit/changelog.md Adds a v14 workbooks changelog entry describing the link fix and updates metadata date.

@flanakin flanakin enabled auto-merge (squash) March 3, 2026 03:07
@flanakin flanakin added Tool: Workbooks Azure Monitor workbooks Tool: Optimization workbook FinOps workbook for cost optimization labels Mar 3, 2026
@flanakin flanakin merged commit 69dfa85 into dev Mar 3, 2026
4 checks passed
@flanakin flanakin deleted the flanakin/fix-workbook-feedback-link branch March 3, 2026 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Micro PR 🔬 Very small PR that should be especially easy for newcomers Needs: Review 👀 PR that is ready to be reviewed Tool: Optimization workbook FinOps workbook for cost optimization Tool: Workbooks Azure Monitor workbooks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Advisor | Workbooks | Cost Optimization submit feedback link error 404

4 participants