Skip to content

Fix autocomplete returning erroneous values#3339

Merged
joe-elliott merged 7 commits intografana:mainfrom
joe-elliott:fix-tag-values
Jan 29, 2024
Merged

Fix autocomplete returning erroneous values#3339
joe-elliott merged 7 commits intografana:mainfrom
joe-elliott:fix-tag-values

Conversation

@joe-elliott
Copy link
Copy Markdown
Collaborator

@joe-elliott joe-elliott commented Jan 26, 2024

What this PR does:
In certain circumstances filtered autocomplete would return incorrect results due to an incorrect iterator scope. The actual fix is this line

  • removed an unused param in createDistinctAttributeOperator
  • added integration tests for filtered autocomplete

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@joe-elliott joe-elliott marked this pull request as ready for review January 26, 2024 18:18
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Copy link
Copy Markdown
Contributor

@mapno mapno left a comment

Choose a reason for hiding this comment

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

LGTM. Nice catch

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.

2 participants