Skip to content

update indexing fields dialog - #443

Open
trean wants to merge 2 commits into
masterfrom
payload-index-new-fields
Open

update indexing fields dialog#443
trean wants to merge 2 commits into
masterfrom
payload-index-new-fields

Conversation

@trean

@trean trean commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Added UI controls in PayloadIndexDialog.jsx to allow users to enable/disable ASCII folding, select a stemmer language, and choose a stopwords language for text fields, as well as enable prefix matching for keyword fields.

@trean
trean requested review from generall and a lite review from Copilot September 7, 2026 16:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The new UI fields, schema/state mapping, helper serialization, and unit tests are consistent with each other and do not introduce verified regressions in the reviewed changes.

Pull request overview

This PR updates the payload indexing UI and supporting helpers to expose additional index parameters for Qdrant payload indexes, allowing users to configure richer behavior when creating or editing payload indexes.

Changes:

  • Add keyword index option for prefix matching.
  • Add text index options for ASCII folding, stemmer language, and stopwords language.
  • Extend payload index form→schema conversion logic and update tests accordingly.
File summaries
File Description
src/components/Points/PayloadIndexDialog.jsx Adds new UI controls and schema↔state mapping for keyword/text index parameters.
src/lib/payload-index-helpers.js Extends form-to-index-config conversion to include new keyword/text params.
src/lib/tests/payload-index-helpers.test.js Updates/extends unit tests to cover new parameters and default/omission behavior.
Review details
  • Files reviewed: 3/3 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@generall

generall commented Sep 9, 2026

Copy link
Copy Markdown
Member

amount of parameters stacked makes modal longer than my screen. Propose to make them more compact or even hide under the "advanced" toggle
image

@generall

generall commented Sep 9, 2026

Copy link
Copy Markdown
Member

Also, new params are missing in create collection dialog
image

@trean

trean commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

@generall

amount of parameters stacked makes the modal longer than my screen. Propose making them more compact or even hiding them under the "advanced" toggle

  • moved fields under "Advanced options". WDYT?

Also, new params are missing in the create collection dialog

  • it was stale in the package-lock.json. Fixed. Also, check if you don't have an old version locally.

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.

3 participants