Skip to content

multiselect field error after 'New' #30

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

Closed
icebob opened this issue Aug 9, 2016 · 20 comments
Closed

multiselect field error after 'New' #30

icebob opened this issue Aug 9, 2016 · 20 comments

Comments

@icebob
Copy link
Member

icebob commented Aug 9, 2016

@lionel-bijaoui could you check this problem?

Reproduction:

  1. npm run dev
  2. Press New button
  3. check console messages. It has 4 error messages, and fieldVueMultiSelect dropdown is empty.
    image
@icebob icebob added the bug label Aug 9, 2016
@icebob icebob changed the title multiselect error after 'New' multiselect field error after 'New' Aug 9, 2016
@lionel-bijaoui
Copy link
Member

I going to check this today.

@icebob
Copy link
Member Author

icebob commented Aug 10, 2016

Thank you!

@lionel-bijaoui
Copy link
Member

Ok so vue-multiselect don't like to have an undefined value and crash.

@icebob
Copy link
Member Author

icebob commented Aug 10, 2016

So is it a multiselect bug? Are we report it?

@lionel-bijaoui
Copy link
Member

I'm raising an issue right now

@icebob
Copy link
Member Author

icebob commented Aug 10, 2016

Great! Thanks!

@lionel-bijaoui
Copy link
Member

shentao/vue-multiselect#80

@icebob
Copy link
Member Author

icebob commented Sep 12, 2016

@lionel-bijaoui
Copy link
Member

We have green light for this

@icebob
Copy link
Member Author

icebob commented Sep 20, 2016

Could you test it in dev?

@lionel-bijaoui
Copy link
Member

Yes

@lionel-bijaoui
Copy link
Member

lionel-bijaoui commented Sep 20, 2016

@icebob I still have a problem. 😞
I have changed some information, clicked save, clicked another profile then clicked back to the original profile and the data where still here. I got no error message in the console.
But when I do new, same error message.
I can't find where the error is happening...
Edit: maybe rawgit is not yet updated ?

@icebob
Copy link
Member Author

icebob commented Sep 20, 2016

I dont think so rawgit cached.
We need to create a fiddle with this issue to monterail, no?

@lionel-bijaoui
Copy link
Member

Well, they have fixed shentao/vue-multiselect#88 but not shentao/vue-multiselect#80

@icebob
Copy link
Member Author

icebob commented Sep 22, 2016

ohh, my mistake. Sorry.

@lionel-bijaoui
Copy link
Member

No pb, I did the same mistake

@icebob icebob added the later label Sep 26, 2016
@icebob
Copy link
Member Author

icebob commented Feb 15, 2017

This problem is alive. I tested. I created a fiddle for developer of multiselect.

https://jsfiddle.net/icebob/0mg1v81e/174/

@lionel-bijaoui
Copy link
Member

They don't like to have an undefined or null value. Maybe we could prevent it ?

@icebob
Copy link
Member Author

icebob commented Feb 15, 2017

But on the website in the "Basic Usage", he uses value: null

@icebob
Copy link
Member Author

icebob commented Mar 29, 2017

Fixed in 2.0.0-beta15

Fiddle

@icebob icebob closed this as completed Mar 29, 2017
@icebob icebob mentioned this issue Mar 29, 2017
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants