Skip to content

docs: update SchemaFieldTypes to SCHEMA_FIELD_TYPE for redis@5.x #2992

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.

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

Merged
merged 1 commit into from
Jun 6, 2025

Conversation

23navi
Copy link
Contributor

@23navi 23navi commented Jun 6, 2025

Description

docs: fix stale SchemaFieldTypes usage for redis@5.x in search docs

Closes: #2985

Redis v5 introduced a breaking change: SchemaFieldTypes was replaced with SCHEMA_FIELD_TYPE [^enum-to-constants].

Updated the documentation in packages/search to reflect the correct usage:

  • For redis@4.x, SchemaFieldTypes is correct
  • For redis@5.x, SCHEMA_FIELD_TYPE must be used

Previous docs were misleading and based on pre-v5 enums. This change ensures accurate guidance across versions.


Checklist

  • Does npm test pass with this change (including linting)?
  • Is the new or changed code fully tested?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

@nkaradzhov
Copy link
Collaborator

@23navi thanks for the PR!

@nkaradzhov nkaradzhov merged commit 21d8f0e into redis:master Jun 6, 2025
2 checks passed
florian-schunk pushed a commit to florian-schunk/node-redis that referenced this pull request Jun 18, 2025
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.

Use of SCHEMA_FIELD_TYPE and SchemaFieldTypes in redisearch
2 participants