Skip to content

Conversation

@brandonc
Copy link
Contributor

@brandonc brandonc commented Oct 31, 2025

Description

🎫 TF-31754

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

@brandonc brandonc requested a review from a team as a code owner October 31, 2025 15:33
@github-actions
Copy link
Contributor

github-actions bot commented Oct 31, 2025

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal ✅ Ready (Inspect) Visit Preview Thu Nov 6 22:12:21 UTC 2025
Unified Docs API ✅ Ready (Inspect) Visit Preview Thu Nov 6 22:05:04 UTC 2025

@github-actions
Copy link
Contributor

github-actions bot commented Oct 31, 2025

Broken Link Checker

No broken links found! 🎉

Copy link

@ctrombley ctrombley left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@trujillo-adam trujillo-adam left a comment

Choose a reason for hiding this comment

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

Just a few minor suggestions.

@brandonc brandonc merged commit a71c0d7 into main Nov 6, 2025
15 checks passed
@brandonc brandonc deleted the TF-31754-document-list-projects-api-tag-filter-parameters branch November 6, 2025 23:06
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.

3 participants