Skip to content

Improve databases documentation#12344

Merged
mladlow merged 3 commits intohashicorp:mainfrom
multani:fix-doc2
Dec 20, 2021
Merged

Improve databases documentation#12344
mladlow merged 3 commits intohashicorp:mainfrom
multani:fix-doc2

Conversation

@multani
Copy link
Copy Markdown
Contributor

@multani multani commented Aug 18, 2021

Fixed a bunch of formatting issues and broken JSON outputs.

Comment thread website/content/api-docs/secret/databases/index.mdx Outdated
Fixed a bunch of formatting issues and broken JSON outputs.
Copy link
Copy Markdown
Contributor

@taoism4504 taoism4504 left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread website/content/api-docs/secret/databases/mssql.mdx Outdated
Comment thread website/content/api-docs/secret/databases/oracle.mdx Outdated
Comment thread changelog/12344.txt Outdated
@pmmukh
Copy link
Copy Markdown
Contributor

pmmukh commented Dec 17, 2021

Thanks for submitting the PR, it looks great! Added a couple comments.

@vercel vercel bot temporarily deployed to Preview – vault-storybook December 17, 2021 20:18 Inactive
Comment thread website/content/api-docs/secret/databases/mssql.mdx Outdated
Comment thread website/content/api-docs/secret/databases/oracle.mdx Outdated
Comment thread website/content/api-docs/secret/databases/postgresql.mdx Outdated
Comment thread website/content/api-docs/secret/databases/redshift.mdx Outdated
Comment thread website/content/api-docs/secret/databases/snowflake.mdx Outdated
@vercel vercel bot temporarily deployed to Preview – vault-storybook December 17, 2021 20:32 Inactive
@multani
Copy link
Copy Markdown
Contributor Author

multani commented Dec 17, 2021

@pmmukh I reverted the changes on <= and remove the changelog file.

@multani multani requested a review from pmmukh December 17, 2021 20:32
Copy link
Copy Markdown
Contributor

@pmmukh pmmukh 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 the changes, lgtm!
There's something wrong with the Vercel check atm, once I figure that out will get this merged in.

@mladlow mladlow merged commit 9601d9c into hashicorp:main Dec 20, 2021
@multani multani deleted the fix-doc2 branch December 20, 2021 20:52
heppu pushed a commit to heppu/vault that referenced this pull request Jan 13, 2022
* Improve databases documentation

Fixed a bunch of formatting issues and broken JSON outputs.

* Remove changelog entry

* Apply suggestions from code review
pull bot pushed a commit to sigtrap/vault that referenced this pull request Feb 13, 2026
Signed-off-by: Ryan Cragun <me@ryan.ec>
Co-authored-by: Ryan Cragun <me@ryan.ec>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants