Skip to content

Conversation

@meirish
Copy link
Contributor

@meirish meirish commented Nov 7, 2025

Description

🎫 IPL-8702 https://hashicorp.atlassian.net/browse/IPL-8702

A redirect that was previously working no longer is. Notably there was a redirect from the source URL to a destination that then was used as a source for another redirect. For whatever reason chaining the redirects in this way does not work (I don't know if this is supposed to be supported or not), so this PR updates the first redirect so the destination matches the second redirect.

Requested review scope:

  • Content touched by the PR only (typos, clarifications, tips)
  • Code test (command and code block changes)
  • Flow and language near changes (new/rearranged steps)
  • Review everything (rewrites, major changes)

Review urgency:

  • ASAP (bug fixes, broken content, imminent releases)
  • 3 days (small changes, easy reviews)
  • 1 week (default)
  • Best effort (very non-urgent)

All updates:

I have:

  • Verified that all status checks have passed
  • Verified that preview environment has successfully deployed
  • Verified appropriate label applied (hcp + product name)
  • Added all required reviewers (code owners and external)

Content checklist (optional)

Please do these things before requesting a review. I have:

  • Made any associated code repositories public
  • Added the hashicorp-education/teamName to any additional code or example repos as repo admin
  • Added redirects for any moved or removed pages
  • Spell checked the tutorial(s)
  • Followed the unified style guide
  • Linted code snippets (Details per language here)
  • Checked the steps for completeness (no steps are implied or hidden)
  • Looked at the local or vercel build and checked each new or changed page for:
    • display on the product curriculum page
    • callout box formatting
    • code block highlighting
    • right-hand navigation
    • next and back buttons
    • URL path

},
{
"source": "/hcp/docs/iam/service-principal",
"destination": "/hcp/docs/hcp/iam/service-principal",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This redirect goes to the page service-principals is also meant to go to.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal ✅ Ready (Inspect) Visit Preview Mon Nov 10 17:25:26 UTC 2025
Unified Docs API ✅ Ready (Inspect) Visit Preview Mon Nov 10 17:18:45 UTC 2025

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

Broken Link Checker

No broken links found! 🎉

@meirish meirish force-pushed the hcp-tf-provider-redirect branch from d65ec1e to 676ba6c Compare November 10, 2025 17:02
Copy link
Contributor

@jonathanfrappier jonathanfrappier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Source 404s, dest works. Thanks for catching this!

@jonathanfrappier jonathanfrappier merged commit 6283aa4 into main Nov 10, 2025
15 checks passed
@jonathanfrappier jonathanfrappier deleted the hcp-tf-provider-redirect branch November 10, 2025 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants