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

Jump to bottom

Wrong definition of the type AsyncComponentFactory #10252

Open
Maorey opened this issue Jul 10, 2019 · 14 comments · May be fixed by #10253
Open

Wrong definition of the type AsyncComponentFactory #10252

Maorey opened this issue Jul 10, 2019 · 14 comments · May be fixed by #10253
Labels
good first issue has PR typescript

Comments

@Maorey
Copy link

@Maorey Maorey commented Jul 10, 2019

Version

2.6.10

Reproduction link

https://github.com/vuejs/vue

Steps to reproduce

  1. compare the AsyncComponentFactory definition with the document

What is expected?

attribute 'component' should be a promise

What is actually happening?

AsyncComponentPromise

I'm looking forward to your solution to this issue
@posva
Copy link
Member

@posva posva commented Jul 10, 2019

It seems the type should be Promise<Component | EsModuleComponent>

@NataliaTepluhina NataliaTepluhina linked a pull request Jul 10, 2019 that will close this issue
Open
13 tasks
@posva posva added the has PR label Jul 10, 2019
NataliaTepluhina added a commit to NataliaTepluhina/vue that referenced this issue Jul 14, 2019
@NataliaTepluhina
fix: add test for AsyncComponentFactory
3be9e28 
Close vuejs#10252
NataliaTepluhina added a commit to NataliaTepluhina/vue that referenced this issue Jul 14, 2019
@NataliaTepluhina
fix: fixed a type for AsyncComponentFactory component
f75a39d 
Close vuejs#10252
@Maorey Maorey closed this as completed Jul 22, 2019
@posva posva reopened this Jul 22, 2019
@harshajayaweeraXHJ
Copy link

@harshajayaweeraXHJ harshajayaweeraXHJ commented Aug 31, 2019

Is this issue still available

@jvik
Copy link

@jvik jvik commented Oct 2, 2019

@harshajayaweeraXHJ The issue already has a PR

@juzhiyuan
Copy link

@juzhiyuan juzhiyuan commented Aug 3, 2021

Any update here?

@tulpauction001

This comment has been minimized.

Sign in to view
@guptadeepak8
Copy link

@guptadeepak8 guptadeepak8 commented Jan 16, 2022

is anyone working on this issue ?

@adamcole123
Copy link

@adamcole123 adamcole123 commented Jan 16, 2022

@guptadeepak8 This issue already has a PR but looks like it still needs to be merged.

@Irtaza1122
Copy link

@Irtaza1122 Irtaza1122 commented Jan 29, 2022

Need help

@SyedAzam246
Copy link

@SyedAzam246 SyedAzam246 commented Feb 9, 2022

@adamcole123 Seems like it, this PR has gone unnoticed.

@tinanazee
Copy link

@tinanazee tinanazee commented Feb 14, 2022

I have the need to understand this proposed of resolution.

@Malak39
Copy link

@Malak39 Malak39 commented Feb 26, 2022

لقد مرت هذي العلاقات العامه لاستطيع

@Mikecarbon
Copy link

@Mikecarbon Mikecarbon commented Mar 17, 2022

Version

2.6.10

Reproduction link

https://github.com/vuejs/vue

Steps to reproduce

  1. compare the AsyncComponentFactory definition with the document

What is expected?

attribute 'component' should be a promise

What is actually happening?

AsyncComponentPromise

I'm looking forward to your solution to this issue

.5

@stupidmikufan
Copy link

@stupidmikufan stupidmikufan commented Mar 19, 2022

i get a lot of those oh and hi

@stupidmikufan
Copy link

@stupidmikufan stupidmikufan commented Mar 19, 2022

if u want me to help then I'll be happy to

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue has PR typescript
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: changed a type for AsyncComponentFactory component(fix #10252) NataliaTepluhina/vue
16 participants
@posva @juzhiyuan @harshajayaweeraXHJ @jvik @Maorey @SyedAzam246 @adamcole123 @tulpauction001 @guptadeepak8 @stupidmikufan @Irtaza1122 @tinanazee @Malak39 @Mikecarbon and others