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.

Sign up for GitHub

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

Jump to bottom

Fix Vault Docs #617

Merged
merged 1 commit into from
Sep 28, 2023
Merged

Fix Vault Docs #617

merged 1 commit into from
Sep 28, 2023

Conversation

aidan-mundy
Copy link
Contributor

🛠️ Description

  • Fixes problems missed/introduced by VAULT-18334: Documentation Subcategories #585
    • Improper header formatting (header displays as part of page text and categorization is ignored)
    • Product-specific guides missing a product category
    • Template missing for hcp_vault_plugin resource and datasource

@aidan-mundy aidan-mundy added bug Something isn't working documentation Improvements or additions to documentation pr/no-changelog Auto-pinning labels Sep 28, 2023
@aidan-mundy aidan-mundy requested a review from a team as a code owner September 28, 2023 17:58
@aidan-mundy aidan-mundy merged commit 7b32fc3 into main Sep 28, 2023
9 checks passed
@aidan-mundy aidan-mundy deleted the aidan-mundy/fix-docs-vault branch September 28, 2023 18:08
@aidan-mundy aidan-mundy mentioned this pull request Oct 3, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AnPucel AnPucel AnPucel approved these changes

@tcrayford tcrayford Awaiting requested review from tcrayford tcrayford is a code owner automatically assigned from hashicorp/vault-cloud

@hashiblaum hashiblaum Awaiting requested review from hashiblaum hashiblaum is a code owner automatically assigned from hashicorp/vault-cloud

Assignees
No one assigned
Labels
bug Something isn't working documentation Improvements or additions to documentation pr/no-changelog Auto-pinning
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aidan-mundy @AnPucel