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

Website: Demo improvements #4

Merged
merged 5 commits into from
Mar 17, 2015
Merged

Website: Demo improvements #4

merged 5 commits into from
Mar 17, 2015

Conversation

pearkes
Copy link
Contributor

@pearkes pearkes commented Mar 17, 2015

This wires in the loading indications, properly handles refocusing inputs and throws the sidebar overlay on the main page when demo is active, as well as a close button.

@pearkes
Copy link
Contributor Author

pearkes commented Mar 17, 2015

Oh, GIF:

foo

pearkes added a commit that referenced this pull request Mar 17, 2015
@pearkes pearkes merged commit 8c93496 into master Mar 17, 2015
@pearkes pearkes deleted the website-demo-updates branch March 17, 2015 20:09
@catsby
Copy link
Contributor

catsby commented Mar 18, 2015

Launch interactive demo

So great

briankassouf pushed a commit that referenced this pull request Jan 17, 2017
jeffwecan added a commit to jeffwecan/vault that referenced this pull request Nov 7, 2018
[SRE-432] Install ansible-modules-hashivault in vault role
swatibourai0560 added a commit to swatibourai0560/vault that referenced this pull request Dec 10, 2024
… role name is consistent.

The role name specified in allowed_roles="my-role" under the section "Rootless Configuration and Password Rotation for Static Roles" should align with the static role name in step hashicorp#3. Previously, the command incorrectly used "my-static-role"; it should be "my-role" to match the earlier step.

The same role name should also be used when reading the static credentials in step hashicorp#4
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.

2 participants