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

Info table row typography #12908

Merged
merged 4 commits into from Oct 22, 2021
Merged

Info table row typography #12908

merged 4 commits into from Oct 22, 2021

Conversation

zofskeez
Copy link
Contributor

This is a small change of font used for the value field in the info table row component by using a span instead of code tag.

image
image

@zofskeez zofskeez added the ui label Oct 22, 2021
@vercel vercel bot temporarily deployed to Preview – vault-storybook October 22, 2021 19:13 Inactive
@vercel vercel bot temporarily deployed to Preview – vault October 22, 2021 19:13 Inactive
@zofskeez zofskeez added this to the 1.9 milestone Oct 22, 2021
@zofskeez
Copy link
Contributor Author

It looks like a possible flaky acceptance test is failing.

tests/acceptance/settings/mount-secret-backend-test.js
version 2 with no update to config endpoint still allows mount of secret engine

It passes locally the majority of the time but I have been able to get it to fail a couple of times. I'll see if I can isolate the issue and refactor it.

@zofskeez
Copy link
Contributor Author

In the test a series of console commands are run, the last of which creates a token which is then accessed with consoleComponent.lastLogOutput. When the test fails the lastLogOutput value is Success! Data written to: sys/policies/acl/kv-v2-degrade which is the result of the previous command. Adding await settled() after running the console commands seems to have fixed it.

Copy link
Contributor

@hashishaw hashishaw left a comment

Choose a reason for hiding this comment

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

🚀

@zofskeez zofskeez merged commit fa0467f into main Oct 22, 2021
@zofskeez zofskeez deleted the ui/table-row-typography branch October 22, 2021 21:16
qk4l pushed a commit to qk4l/vault that referenced this pull request Feb 4, 2022
* updates info table row value font

* adds changelog

* updates tests

* attempts to fix flaky mount-secret-backend acceptance test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants