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

Bug fix for issue 4187 (recursive components) (Vue 1.x) #4739

Open
wants to merge 9 commits into
base: 1.0
Choose a base branch
from
Open

Bug fix for issue 4187 (recursive components) (Vue 1.x) #4739

wants to merge 9 commits into from

Conversation

KingMario
Copy link
Contributor

@KingMario KingMario commented Jan 18, 2017

Hopefully fixed the issue #4187. Review needed.

@posva
Copy link
Member

posva commented Jan 18, 2017

Hey, thanks for the PR!
Could you add a test to it, please?

@KingMario
Copy link
Contributor Author

KingMario commented Jan 18, 2017

@posva I'll do it tonight (GMT+8).

@posva
Copy link
Member

posva commented Jan 18, 2017

@KingMario No worries 🙂 Thanks!

@KingMario
Copy link
Contributor Author

Will deal with the unit test error tomorrow. Hold on.

@KingMario
Copy link
Contributor Author

KingMario commented Jan 19, 2017

Done

@KingMario KingMario changed the title Bug fix for issue 4187 (recursive components) Bug fix for issue 4187 (recursive components) Vue 1.x Jan 20, 2017
@KingMario KingMario changed the title Bug fix for issue 4187 (recursive components) Vue 1.x Bug fix for issue 4187 (recursive components) (Vue 1.x) Jan 20, 2017
@yyx990803 yyx990803 added the 1.x label Feb 12, 2017
@cseufert
Copy link

Is there any reason this has not been merged?

@KingMario
Copy link
Contributor Author

@cseufert From the official site, the version plan for Vue 1 is in a half-year interval. So I guess this will not be merged till the day Vue 1.2 is released.

You may just modify your file as in this pull request in node_modules if it's an urgency.

@yyx990803 yyx990803 removed the 1.x label Apr 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants