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

Backport of Secret Metadata Breadcrumb Bug into release/1.13.x #19716

Conversation

hc-github-team-secure-vault-core
Copy link
Collaborator

Backport

This PR is auto-generated from #19703 to be assessed for backporting due to the inclusion of the label backport/1.13.x.

The below text is copied from the body of the original PR.


It was discovered that a 404 was thrown when navigating back a level from the metadata tab of a secret using the breadcrumbs. The link was pointing to the show route in the metadata view rather than list route as in secret tab.

Before:

Screen.Recording.2023-03-22.at.4.21.48.PM.mov

After:

Screen.Recording.2023-03-22.at.4.23.29.PM.mov

Overview of commits

@hc-github-team-secure-vault-core hc-github-team-secure-vault-core force-pushed the backport/ui/VAULT-14286/kvv2-breadcrumb-bug/vastly-arriving-kit branch from 487fd7e to ce9c436 Compare March 23, 2023 16:26
@hc-github-team-secure-vault-core hc-github-team-secure-vault-core force-pushed the backport/ui/VAULT-14286/kvv2-breadcrumb-bug/vastly-arriving-kit branch from 7c8b660 to 52952ef Compare March 23, 2023 16:26
@zofskeez zofskeez enabled auto-merge (squash) March 23, 2023 16:39
@zofskeez zofskeez added this to the 1.13.1 milestone Mar 23, 2023
@zofskeez zofskeez added the bug Used to indicate a potential bug label Mar 23, 2023
@zofskeez zofskeez merged commit 4472e4a into release/1.13.x Mar 23, 2023
@Monkeychip Monkeychip deleted the backport/ui/VAULT-14286/kvv2-breadcrumb-bug/vastly-arriving-kit branch March 23, 2023 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Used to indicate a potential bug ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants