Skip to content

Is there a better way of handling v-model? #22

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
afontcu opened this issue May 13, 2019 · 3 comments
Closed

Is there a better way of handling v-model? #22

afontcu opened this issue May 13, 2019 · 3 comments

Comments

@afontcu
Copy link
Member

afontcu commented May 13, 2019

Hi! I know that #16 has been closed and that there's an answer to the original question, but I was wondering if there's any advance in the idea of a fireEvent.update method.

I guess it'd make sense to come up with a way of handling input data consistently, without depending on v-model being there... am I right? For that to happen, should the issue mentioned here be fixed? By the way, is there an open issue in the official library repo? Failed to find any reference.

Thanks, and thank you for your work with this library. Testing is pleasant again.

@dfcook
Copy link
Collaborator

dfcook commented Jun 1, 2019

Finally got around to looking at this, PR raised.

Assuming this goes in, should updateState be removed?

@afontcu
Copy link
Member Author

afontcu commented Jun 1, 2019

Assuming this goes in, should updateState be removed?

Makes sense! It is quite pointless now, isn't it? Its only purpose AFAIK was to make v-model work.

@dfcook
Copy link
Collaborator

dfcook commented Jun 1, 2019

Fixed in #34

@dfcook dfcook closed this as completed Jun 1, 2019
dfcook added a commit that referenced this issue Jun 1, 2019
Add update method for model updates
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

No branches or pull requests

2 participants