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

defineProps bug when Interface properteis contain two or more : colons #8910

Closed
cicin411 opened this issue Aug 4, 2023 · 0 comments · Fixed by #8654
Closed

defineProps bug when Interface properteis contain two or more : colons #8910

cicin411 opened this issue Aug 4, 2023 · 0 comments · Fixed by #8654
Labels
🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. has PR A pull request has already been submitted to solve the issue 🐞 bug Something isn't working scope: compiler

Comments

@cicin411
Copy link

cicin411 commented Aug 4, 2023 •
edited
Loading

Vue version

3.3

Link to minimal reproduction

https://play.vuejs.org/#eNqtU1Fv0zAQ/iuHX9JKoxXsLaRFgCoBEjCxIR4wmrz4mnk4Z8t2uk5R/juXpGuRNvWBLU+57/vO+b7cuRXvvJ9tGhS5KGIZjE8QMTUerKJqIUWKUiwlmdq7kKCFgGvoYB1cDRm3ZUxRwrBWJcKFurJ4FpyPklpJwA9qk3o0hyvnLCp6M+KlKq9Rf3H6AZM5+uG1Spjj1ivSqLO3OUwIb1e7OmcTSjuyd6Do7tfvKSyWsHFGPzghosUyHU4439XHTugkPZ7p9XOEunxyqMv/DiWpdBQT+D7NAjSuDY3RikPK5WTK+oO4jhUs+qlPso9orYOfLlj9IptKKubjwvB6cJGw9pb9cQVQXL9atu3Q3HXFnKsBNeSbBJuXNf8iy8vF/LBdxXzfLU545fjTa1PNbqIj3svhr0tRutobi+GbT4atSZHDbh5SKHZ2+3nAUmjw5B7neZR/HsFv4rbHpDgLGDFsUIo9l1SoMI306vwrbvl9T7LzxrL6CPkdo7NN73GUvW9Is+1/dIPbT8ONMlRdxNU2IcX7UL3RXtkNein4ln04Ev1g93R2OvTxuEX3F/4cTSM=

Steps to reproduce

export interface ccTableProps
{
editable: boolean;
cachedMode: boolean;
'onUpdate:expanded'?: (newExpanded: readonly any[]) => void;
'onUpdate:selected'?: (newSelected: readonly any[]) => void;
}

defineProps

What is expected?

define props correctly

What is actually happening?

vite error with tips about missing xxx
2023-08-04_092136

System Info

No response

Any additional comments?

interface properties contains
one colon: nothing wrong.
two or more colons: vite error.
@cicin411 cicin411 changed the title defineProps bug when Interface property contains two or more : colons defineProps bug when Interface properteis contain two or more : colons Aug 4, 2023
@edison1105 edison1105 mentioned this issue Aug 4, 2023
Closed
@edison1105 edison1105 added 🐞 bug Something isn't working scope: compiler 🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. labels Aug 4, 2023
@sxzz sxzz added the has PR A pull request has already been submitted to solve the issue label Aug 12, 2023
@auvred auvred mentioned this issue Aug 13, 2023
Merged
@danielbender1989 danielbender1989 mentioned this issue Sep 20, 2023
Closed
yyx990803 pushed a commit that referenced this issue Nov 10, 2023
@auvred
fix(compiler-sfc): consistently escape type-only prop names (#8654)
3e08d24 
close #8635
close #8910
close vitejs/vite-plugin-vue#184
@github-actions github-actions bot locked and limited conversation to collaborators Nov 25, 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. has PR A pull request has already been submitted to solve the issue 🐞 bug Something isn't working scope: compiler
Projects
None yet
Milestone
No milestone
3 participants
@edison1105 @sxzz @cicin411