Skip to content

Conversation

ldouillet
Copy link
Contributor

Description

Fix a broken link in the Web REPL help commands which is pointing to https://developer.hashicorp.com/vault/docs/commands/web (broken)
image

Changed to https://developer.hashicorp.com/vault/docs/ui/web-cli

Copy link

hashicorp-cla-app bot commented Apr 3, 2025

CLA assistant check
All committers have signed the CLA.

@ldouillet ldouillet marked this pull request as ready for review April 3, 2025 15:28
@ldouillet ldouillet requested review from a team as code owners April 3, 2025 15:28
@ldouillet ldouillet requested a review from rculpepper April 3, 2025 15:28
@@ -25,6 +25,6 @@ Web REPL Commands:

</pre>
<p class="console-ui-panel-intro is-font-mono has-bottom-margin-s">
For more detailed documentation, see the <Hds::Link::Inline @href={{doc-link "/vault/docs/commands/web"}}>HashiCorp Developer site</Hds::Link::Inline>.
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for fixing! I was wondering why the redirect was working appropriately here and noticed there that command is singular and here it's plural

Copy link
Contributor

@hellobontempo hellobontempo left a comment

Choose a reason for hiding this comment

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

Thank you for fixing! Just a small typo fix in the changelog then it's good to go 🚀

hellobontempo
hellobontempo previously approved these changes Apr 7, 2025
@emoncuso emoncuso removed the request for review from rculpepper April 15, 2025 16:10
@hellobontempo
Copy link
Contributor

It looks like there's a linting error so tests haven't run - are you able to resolve on your end?

@anwittin anwittin modified the milestones: 1.19.2, 1.19.3 Apr 18, 2025
@anwittin anwittin removed this from the 1.19.3 milestone Apr 29, 2025
@anwittin anwittin added this to the 1.19.4 milestone Apr 29, 2025
@Monkeychip
Copy link
Contributor

It looks like there's a linting error so tests haven't run - are you able to resolve on your end?

@ldouillet would love to get this in before the next patch release. Are you able to address the linting error mentioned above?

@Monkeychip Monkeychip changed the base branch from main to ui/VAULT-35724/broken-link May 19, 2025 14:51
@Monkeychip
Copy link
Contributor

I'm going to merge this into a sidebranch so I can update the formatting issues.

@Monkeychip Monkeychip merged commit 570b2cb into hashicorp:ui/VAULT-35724/broken-link May 19, 2025
1 of 2 checks passed
Monkeychip added a commit that referenced this pull request May 19, 2025
* formatting

* Fix broken link in the Web REPL help commands (#30167)

* fix broken link in the Web REPL Commands help

change from https://developer.hashicorp.com/vault/docs/commands/web (broken) to  https://developer.hashicorp.com/vault/docs/ui/web-cli

* Create changelog for #29013

* change log for pull request #30167

* fix broken link in the Web REPL Commands help

change from https://developer.hashicorp.com/vault/docs/commands/web (broken) to  https://developer.hashicorp.com/vault/docs/ui/web-cli

* Create changelog for #29013

* Delete changelog/29013.txt

* Create 29013.txt

* fix changelog name to align to the PR ID

* remove trailing cr (fix linting error)

---------

Co-authored-by: Violet Hynes <violet.hynes@hashicorp.com>

* Update 30167.txt

* Update log-help.hbs

* Update log-help.hbs

* try this way

* fix changelog

---------

Co-authored-by: Laurent Douillet <69690045+ldouillet@users.noreply.github.com>
Co-authored-by: Violet Hynes <violet.hynes@hashicorp.com>
erentantekin pushed a commit that referenced this pull request May 23, 2025
* formatting

* Fix broken link in the Web REPL help commands (#30167)

* fix broken link in the Web REPL Commands help

change from https://developer.hashicorp.com/vault/docs/commands/web (broken) to  https://developer.hashicorp.com/vault/docs/ui/web-cli

* Create changelog for #29013

* change log for pull request #30167

* fix broken link in the Web REPL Commands help

change from https://developer.hashicorp.com/vault/docs/commands/web (broken) to  https://developer.hashicorp.com/vault/docs/ui/web-cli

* Create changelog for #29013

* Delete changelog/29013.txt

* Create 29013.txt

* fix changelog name to align to the PR ID

* remove trailing cr (fix linting error)

---------

Co-authored-by: Violet Hynes <violet.hynes@hashicorp.com>

* Update 30167.txt

* Update log-help.hbs

* Update log-help.hbs

* try this way

* fix changelog

---------

Co-authored-by: Laurent Douillet <69690045+ldouillet@users.noreply.github.com>
Co-authored-by: Violet Hynes <violet.hynes@hashicorp.com>
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.

5 participants