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

UI/fix kvv2 version #11258

Merged
merged 5 commits into from
Apr 2, 2021
Merged

UI/fix kvv2 version #11258

merged 5 commits into from
Apr 2, 2021

Conversation

chelshaw
Copy link
Contributor

@chelshaw chelshaw commented Apr 1, 2021

Unselected dropdown option caused a bug when enabling a v2 secrets engine and creating a secret, where the UI didn't recognize that the new secrets engine was version 2 until a refresh caused a new network request. In addition to correctly setting the default kv version to 2, we moved that option under the Method Options dropdown:

Version selection is in Method Options toggle

This PR also updates the MessageError component to better handle the type of error that was resulting in a message like [object Object], so that arrays of objects are instead passed into the component as strings

Better error message than [object Object]

@chelshaw chelshaw added the ui label Apr 1, 2021
@chelshaw chelshaw added this to the 1.7.1 milestone Apr 1, 2021
@chelshaw chelshaw added bug Used to indicate a potential bug backport labels Apr 1, 2021
@vercel vercel bot temporarily deployed to Preview – vault April 2, 2021 16:06 Inactive
Copy link
Contributor

@Monkeychip Monkeychip left a comment

Choose a reason for hiding this comment

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

Thanks for tackling this!

@vercel vercel bot temporarily deployed to Preview – vault-storybook April 2, 2021 16:15 Inactive
@chelshaw chelshaw merged commit 8cdcd5c into master Apr 2, 2021
@chelshaw chelshaw deleted the ui/fix-kvv2-version branch April 2, 2021 20:17
chelshaw added a commit that referenced this pull request Apr 2, 2021
* Update default form values for kv

* Group kv version option in 'Method Options' group

* Fix tests, explicitly set if select input does not have default

* Handle array of objects from adapterError.errors in MessageError component

* Add changelog
chelshaw added a commit that referenced this pull request Apr 2, 2021
AndreyZamyslov pushed a commit to yandex-cloud/vault that referenced this pull request Jun 10, 2021
* Update default form values for kv

* Group kv version option in 'Method Options' group

* Fix tests, explicitly set if select input does not have default

* Handle array of objects from adapterError.errors in MessageError component

* Add changelog
jartek pushed a commit to jartek/vault that referenced this pull request Sep 11, 2021
* Update default form values for kv

* Group kv version option in 'Method Options' group

* Fix tests, explicitly set if select input does not have default

* Handle array of objects from adapterError.errors in MessageError component

* Add changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport bug Used to indicate a potential bug ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants