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.

Sign up for GitHub

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

Jump to bottom

DEV Warning: type check failed for enum prop #7511

Closed
dondevi opened this issue Jan 12, 2023 · 1 comment · Fixed by #7548
Closed

DEV Warning: type check failed for enum prop #7511

dondevi opened this issue Jan 12, 2023 · 1 comment · Fixed by #7548
Labels
🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. scope: compiler

Comments

@dondevi
Copy link

dondevi commented Jan 12, 2023 •
edited
Loading

Vue version

3.2.45

Link to minimal reproduction

https://sfc.vuejs.org/#__DEV__eNqVVE1v00AQ/SsjX5xKiU0LXExailAlDiBVAnGpe3DiSTD1fmh3nVIF/3dmduzUbSqkXmzPvjczz29nd598sjbbdZgUyTKgsm0V8KLUAMvPRlngL4DdQpka2/My2VVth2Uiy5edrYleRHARIaLIonByKrXMJ3Up9GvX2ABtpbfEDr5MwGPoLKGNssYF2IPDDfSwcUZBSuLSDwcsqhIgyzlg8YyXem20DxBlwDmXmJ3E9U2n16ExGkTZjBgnsGd5nGFazFqzjatE71mwSCRByTyRvgtV2ey3N5p8iqnlAJD8QorxGmnhuEx+hWB9keedtnfbbG1UfklY7jodGoWL2qjLt9lZ9u59Xjc+TNcz9Gqxcubeo6OOZTKfFM9pcYdu4VDX6ND9t9kz7pOGz7CjptyTrOjJgNHk4wnx2OI6iLxi3H7rjPVZHImf02EpVNeGxrbMGT8Pg9Ro2wUCjI5fsh6bUBtjefck4FncGEfUhrRAo0HQQXJsdIcPA549GdeJyBdBMqZatViP+BiPlEEPwH4PkdBWK2yhJ5+izlykiDfkMJvzuvlH3Sm44kecKKN5kNPTNI5AuDccnXFELelE6IBuU60RrtlzyXk0/mMhpf7G180tDTfBg/MErgyNfqVl5sfTE3eP2tS4aTTGusv4vIiHSUiomnDgXFHgZzfp8V2Q3sr5k6RhnyjvhoXs5aAWkKZz8oKMpM9rEuSR7GDraH3cgQKCo0PdRx8OmfxbGXn0mE9B+iKLvHtkUSAstmRyPQzDN8Md6kCJ/JpcFHwvyeXS009EUhYqt8UAlYcvP759/R51X7WoCIt2s1Ozl6yZS6njG4cHJDy0csLEiPF6sVVdN3pbwOkb+yeWl9SBnvT/AKhB+OM=

Steps to reproduce

  1. Select a value
  2. Console warning: Invalid prop: type check failed for prop "modelValue". Expected Null | Array

What is expected?

No warning.

What is actually happening?

image

Props Vue Check enum
enum null Skip check
enum | enum[] [null, Array] Check for null then warning

System Info

No response

Any additional comments?

No response
@sxzz sxzz mentioned this issue Jan 17, 2023
Merged
@sxzz sxzz added the 🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. label Feb 17, 2023
@yyx990803 yyx990803 reopened this Mar 28, 2023
@yyx990803
Copy link
Member

fix in 2e074a7 was incorrect.

IAmSSH pushed a commit to IAmSSH/core that referenced this issue May 14, 2023
@yyx990803 @IAmSSH
fix(compiler-sfc): infer correct type for enums
3083dd0 
fix vuejs#7511
IAmSSH pushed a commit to IAmSSH/core that referenced this issue May 14, 2023
@yyx990803 @IAmSSH
fix(compiler-sfc): properly handle unknown types in runtime prop infe…
3ad7f5b 
…rence

fix vuejs#7511
@github-actions github-actions bot locked and limited conversation to collaborators Sep 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. scope: compiler
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(runtime-core): add skipCheck for prop vuejs/core
4 participants
@yyx990803 @dondevi @sxzz @zhangzhonghe