Skip to content

website: Add Qualified chatbot to consent manager#13768

Merged
alexcarpenter merged 1 commit intomainfrom
nq.web/add-qualified-chatbot-consent
Jan 26, 2022
Merged

website: Add Qualified chatbot to consent manager#13768
alexcarpenter merged 1 commit intomainfrom
nq.web/add-qualified-chatbot-consent

Conversation

@EnMod
Copy link
Copy Markdown
Contributor

@EnMod EnMod commented Jan 24, 2022

🔍 Preview Link


This PR adds the Qualified chatbot to the Vault website's consent manager.

@EnMod EnMod added website Code changes specifically for the docs website backport/website pr/no-changelog labels Jan 24, 2022
@EnMod EnMod changed the title website: Add Qualified chatbot website: Add Qualified chatbot to consent manager Jan 24, 2022
@EnMod EnMod marked this pull request as ready for review January 24, 2022 19:01
@EnMod EnMod requested review from alexcarpenter and mwickett and removed request for mwickett January 24, 2022 19:02
Copy link
Copy Markdown
Contributor

@alexcarpenter alexcarpenter left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏼

@alexcarpenter alexcarpenter merged commit b204fdd into main Jan 26, 2022
pbromb pushed a commit to pbromb/vault that referenced this pull request Apr 13, 2026
…hicorp#13823)

* add freshest crl to base

* add test

* add helper, add test case for delta crl

* add openssl test

* add changelog

* add removed nil check

* add go doc

* change keytype to ec

* rotate CRL instead of role/issue/revoke a cert and add ldap url test case

* move root generation outside test loop

* remove length check so urls are always set for each test case

* remove unnecessary clearing

Co-authored-by: claire bontempo <68122737+hellobontempo@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr/no-changelog website Code changes specifically for the docs website

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants