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
[FIX] TextValueProvider: duplicate values in dropdown list #3843
Conversation
d00a388
to
926668a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
a6b22af
to
8f92f69
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small comment then we're good :)
}); | ||
|
||
test("Duplicate values will be removed before sending proposals to the autocomplete dropdown in data validation with list", async () => { | ||
addDataValidation(model, "A1", "id", { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also test that the data validation plugin has the filtered values since you filter them in the plugin now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added a test for the plugin, thanks!
Previously, when creating data validation with either a list of values or a range of values, it resulted in an error being thrown by the auto-complete dropdown. This occurred because the values were being used as keys, and by definition, having duplicate keys in a list is nonsensical. This commit addresses the problem by eliminating duplicate values from the list. TaskID: 3768392
8f92f69
to
088e01f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@robodoo r+
Description:
Previously, when creating data validation with either a list of values or a range of values, it resulted in an error being thrown by the auto-complete dropdown. This occurred because the values were being used as keys, and by definition, having duplicate keys in a list is nonsensical.
This PR addresses the problem by eliminating duplicate values from the list.
Task: : 3768392
review checklist