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

fix(runtime-core): bind default function of inject to instance (fix #3923) #3925

Merged
merged 1 commit into from
Jun 9, 2021

Conversation

eunjae-lee
Copy link
Contributor

Summary

This PR fixes the issue #3923.

I added a regression test. I'm not familiar with this code base especially how you normally type things.

Let me know if you need me to modify anything.

}
},
render() {
// @ts-ignore
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know how to make this work without this ts-ignore. Any idea?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's ok for the test case.

@yyx990803 yyx990803 merged commit db1dc1c into vuejs:master Jun 9, 2021
eunjae-lee added a commit to algolia/vue-instantsearch that referenced this pull request Jun 24, 2021
This reverts commit cf7f20a,
because the issue has been fixed since Vue@3.1.2 by
vuejs/core#3925
Haroenv pushed a commit to algolia/instantsearch that referenced this pull request Dec 28, 2022
…instantsearch#951)"

This reverts commit 8348d3d,
because the issue has been fixed since Vue@3.1.2 by
vuejs/core#3925
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.

None yet

2 participants