Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add hvd-docs redirects functionality #2671

Merged
merged 5 commits into from
Jan 22, 2025
Merged

Conversation

LeahMarieBush
Copy link
Contributor

@LeahMarieBush LeahMarieBush commented Jan 21, 2025

πŸ”— Relevant links

πŸ—’οΈ What

This PR adds redirects functionality for hvd-docs. It works the same way as other product redirects in dev-portal.

🀷 Why

There is not currently redirect support for hvd-docs. That team has requested the functionality (see Slack thread)

πŸ§ͺ Testing

  1. Go to /validated-designs/boundary-operating-guides-adoption
  2. Verify that it is not going to that page and instead redirects to the /validated-designs homepage

πŸ’­ Anything else?

The PR has a sha value for the hvd-docs repo that is just for testing. The sha links to a commit that I created on the hvd-docs repo to add an example redirect for testing (See this commit) After PR has been tested by a reviewer, I will change that back to 'main' before merging. See PR comment for more info.

Copy link

vercel bot commented Jan 21, 2025

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
dev-portal βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Jan 22, 2025 5:21pm

// hvd-docs is not hosted on the content API, so we need to use main as the latest sha
const latestContentSha =
repoName === 'hvd-docs'
? 'daf97dfd494470e27acf2b74a95919ac748a54f6'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TODO before merge: change this SHA to 'main'. Currently this a sha from a commit I made in the hvd-docs repo that is used to test this PR

Copy link

github-actions bot commented Jan 21, 2025

πŸ“¦ Next.js Bundle Analysis

This analysis was generated by the next.js bundle analysis action πŸ€–

This PR introduced no changes to the javascript bundle πŸ™Œ

@LeahMarieBush LeahMarieBush marked this pull request as ready for review January 21, 2025 17:41
@LeahMarieBush LeahMarieBush requested review from a team, rmainwork and RubenSandwich and removed request for a team January 21, 2025 17:41
Copy link
Collaborator

@RubenSandwich RubenSandwich left a comment

Choose a reason for hiding this comment

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

Looks good. πŸ‘πŸ»

@LeahMarieBush LeahMarieBush merged commit c9a06fe into main Jan 22, 2025
9 checks passed
@LeahMarieBush LeahMarieBush deleted the leah/feat/hvd-redirects branch January 22, 2025 17:28
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.

3 participants