Skip to content

doc: updating "How to choose a DocumentStore" documentation#11512

Open
davidsbatista wants to merge 10 commits into
mainfrom
doc/update-document-store-page
Open

doc: updating "How to choose a DocumentStore" documentation#11512
davidsbatista wants to merge 10 commits into
mainfrom
doc/update-document-store-page

Conversation

@davidsbatista

Copy link
Copy Markdown
Contributor

Checklist

  • I have read the contributors guidelines and the code of conduct.
  • I have updated the related issue with new insights and changes.
  • I have added unit tests and updated the docstrings.
  • I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test: and added ! in case the PR includes breaking changes.
  • I have documented my code.
  • I have added a release note file, following the contributors guidelines.
  • I have run pre-commit hooks and fixed any issue.

@vercel

vercel Bot commented Jun 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
haystack-docs Ready Ready Preview, Comment Jun 9, 2026 3:31pm

Request Review

@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Coverage report

This PR does not seem to contain any modification to coverable code.

@davidsbatista davidsbatista added type:documentation Improvements on the docs ignore-for-release-notes PRs with this flag won't be included in the release notes. and removed topic:build/distribution labels Jun 5, 2026
@davidsbatista davidsbatista marked this pull request as ready for review June 5, 2026 12:10
@davidsbatista davidsbatista requested a review from a team as a code owner June 5, 2026 12:10
@davidsbatista davidsbatista requested review from kacperlukawski and sjrl and removed request for a team June 5, 2026 12:10
@davidsbatista

Copy link
Copy Markdown
Contributor Author

@claude

claude Bot commented Jun 5, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@davidsbatista

Copy link
Copy Markdown
Contributor Author

@kacperlukawski the image still needs to be improved - I'm working on it

@kacperlukawski kacperlukawski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I left some comments, but overall liked the narrative. I think it would make sense to make each "Best for" paragraph stand out.

Comment thread docs-website/docs/concepts/document-store/choosing-a-document-store.mdx Outdated
Comment thread docs-website/docs/concepts/document-store/choosing-a-document-store.mdx Outdated
Comment thread docs-website/docs/concepts/document-store/choosing-a-document-store.mdx Outdated
Comment thread docs-website/docs/concepts/document-store/choosing-a-document-store.mdx Outdated
…store.mdx

Co-authored-by: Kacper Łukawski <kacperlukawski@users.noreply.github.com>
…store.mdx

Co-authored-by: Kacper Łukawski <kacperlukawski@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore-for-release-notes PRs with this flag won't be included in the release notes. topic:build/distribution type:documentation Improvements on the docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants