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

Wrong definition of the type AsyncComponentFactory #10252

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

Wrong definition of the type AsyncComponentFactory #10252

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

Comments

@Maorey
Copy link

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 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

Is this issue still available

@jvik
Copy link

jvik commented Oct 2, 2019

@harshajayaweeraXHJ The issue already has a PR

@juzhiyuan
Copy link

Any update here?

@tulpauction001

This comment has been minimized.

Sign in to view
@guptadeepak8
Copy link

is anyone working on this issue ?

@adamcole123
Copy link

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

@Irtaza1122
Copy link

Need help

@SyedAzam246
Copy link

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

@tinanazee
Copy link

I have the need to understand this proposed of resolution.

@Malak39
Copy link

Malak39 commented Feb 26, 2022

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

@Mikecarbon
Copy link

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

i get a lot of those oh and hi

@stupidmikufan
Copy link

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

@DaZuiZui
Copy link

Is this issue still available?

@5z5
Copy link

5z5 commented Feb 11, 2023

Is this issue still available
vue@2.6.10 Is it not maintained now, so what is the point of solving this issue?

@StephanDrawe
Copy link

Is this issue still available?

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