Skip to content
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

listPicker always return "test" when multiple allowed #458

Closed
s1nce opened this issue Jan 12, 2020 · 3 comments
Closed

listPicker always return "test" when multiple allowed #458

s1nce opened this issue Jan 12, 2020 · 3 comments
Labels
status:fixed-next-drop Issue will be fixed in upcoming release. type:bug
Milestone

Comments

@s1nce
Copy link

s1nce commented Jan 12, 2020

Category

[ ] Enhancement

[x] Bug

[ ] Question

Version

Please specify what version of the library you are using: [ 1.16.0 ]

Expected / Desired Behavior / Question

List picker should return the list of chosen or empty array or null when multiple is allowed

Observed Behavior

"test is always included as the first element from return even no item is chosen

Steps to Reproduce

It only happens when multiSelect is enabled

<ListPicker
                    context={this.props.context}
                    multiSelect={true}
                    label='Select your list(s)'
                    includeHidden={false}
                    onSelectionChanged={(selected) => console.log(selected)} />

Thanks!

@ghost
Copy link

ghost commented Jan 12, 2020

Thank you for reporting this issue. We will be triaging your incoming issue as soon as possible.

@ghost ghost added the Needs: Triage 🔍 label Jan 12, 2020
@github-actions
Copy link

Thank you for submitting your first issue to this project.

AJIXuMuK added a commit that referenced this issue Jan 15, 2020
#458 listPicker always return "test" when multiple allowed
@AJIXuMuK AJIXuMuK added this to the 1.17.0 milestone Jan 15, 2020
@AJIXuMuK AJIXuMuK added status:fixed-next-drop Issue will be fixed in upcoming release. type:bug and removed Needs: Triage 🔍 labels Jan 15, 2020
@AJIXuMuK
Copy link
Collaborator

Hi @s1nce!
Thanks to @mgwojciech it has been fixed and will be available in the next version.
You can also test it in current beta version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:fixed-next-drop Issue will be fixed in upcoming release. type:bug
Projects
None yet
Development

No branches or pull requests

3 participants