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

[compat] model configuration doesn't support modelValue as a prop value #3771

Closed
CyberAP opened this issue May 14, 2021 · 0 comments · Fixed by #3772
Closed

[compat] model configuration doesn't support modelValue as a prop value #3771

CyberAP opened this issue May 14, 2021 · 0 comments · Fixed by #3772
Labels
🐞 bug Something isn't working scope: v2 compat

Comments

@CyberAP
Copy link
Contributor

CyberAP commented May 14, 2021

Version

3.1.0-beta.3

Reproduction link

https://codesandbox.io/s/frosty-haze-zb9lx?file=/src/Child.vue

Steps to reproduce

  1. Create a component with a model config that supports v-model directive
  2. Define model config prop value as modelValue
  3. Use that component with a v-model directive in a parent component

What is expected?

A prop option should be used to pass model value from parent to child component on a correct prop

What is actually happening?

No model value is passed to the child component

If a prop model config value is changed to any other value it works back again: https://codesandbox.io/s/upbeat-tdd-jypn0?file=/src/Child.vue

A workaround is just to switch to Vue 3 mode.

@CyberAP CyberAP changed the title [compat] model prop option is not considered when using v-model [compat] model configuration doesn't support modelValue as a prop value May 14, 2021
@HcySunYang HcySunYang added 🐞 bug Something isn't working scope: v2 compat labels May 14, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Oct 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐞 bug Something isn't working scope: v2 compat
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants