Skip to content
This repository has been archived by the owner on Jul 5, 2023. It is now read-only.

Remove editions from nav #2493

Merged
merged 1 commit into from
May 24, 2023
Merged

Remove editions from nav #2493

merged 1 commit into from
May 24, 2023

Conversation

mwickett
Copy link
Collaborator

What

Removes the "Editions" menu item. CLI was pointing to the homepage, and Cloud & Enterprise both now go to https://www.hashicorp.com/products/terraform so the whole entry is redundant.

Why

Simplifies the nav for users

@vercel
Copy link

vercel bot commented May 24, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
terraform-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 24, 2023 9:06pm

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think that this file is not used any longer since the nav data was hardcoded into Subnav props (see other changed file), but I left this in just for the sake of completeness.

@mwickett mwickett requested review from a team and nandereck and removed request for a team May 24, 2023 20:50
Copy link
Contributor

@nandereck nandereck left a comment

Choose a reason for hiding this comment

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

LGTM! 🔪

@mwickett mwickett merged commit dcf7cbd into master May 24, 2023
3 checks passed
@mwickett mwickett deleted the mw.remove-editions-nav branch May 24, 2023 21:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants