Skip to content

Conversation

skwowet
Copy link
Member

@skwowet skwowet commented Sep 1, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at a240668

Added support for nested field queries in Opensearch. Modified FieldTranslator and OpensearchQueryParser classes to detect and handle nested fields in the filters object.

🤖 Generated by Copilot at a240668

Nested query added
To filter crowd fields with depth
Autumn leaves nested

Why

How

🤖 Generated by Copilot at a240668

  • Add support for nested field queries in Opensearch (link, link)
  • Create a new method isNestedField in fieldTranslator.ts to check if a field name starts with nested_ (link)
  • Modify parseCrowdFilters in opensearchQueryParser.ts to wrap the query condition for nested fields in a nested query with the correct path and query (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screehshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@skwowet skwowet added the Bug Created by Linear-GitHub Sync label Sep 1, 2023
@skwowet skwowet self-assigned this Sep 1, 2023
@skwowet skwowet requested a review from themarolt September 1, 2023 07:30
@skwowet skwowet merged commit a3399a2 into main Sep 1, 2023
@skwowet skwowet deleted the bugfix/identities branch September 1, 2023 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants