Skip to content
This repository has been archived by the owner on Dec 30, 2022. It is now read-only.

feat(ssr): forward propsData to recreated component #865

Merged
merged 2 commits into from
Sep 30, 2020

Conversation

Haroenv
Copy link
Contributor

@Haroenv Haroenv commented Sep 23, 2020

propsData isn't valid as argument to .extend, but you don't need to call new Vue(comp), it's the same as calling new comp({propsData...})

Base automatically changed from fix/vue-vuex-ssr to master September 24, 2020 07:35
TODO: this throws a warning (but works) because of:

https://github.com/vuejs/vue/blob/7912f75c5eb09e0aef3e4bfd8a3bb78cad7540d7/src/core/util/options.js#L34-L44

I'm not fully sure how to do `new Vue` or pass this parameter correctly. If I `Object.assign` it for `app = new Vue({...extended, propsData})`, it also doesn't work :/
co-authored-by: Eunjae Lee <eunjae-lee@users.noreply.github.com>
@Haroenv Haroenv marked this pull request as ready for review September 29, 2020 14:55
Copy link
Contributor

@eunjae-lee eunjae-lee left a comment

Choose a reason for hiding this comment

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

Looks good to me

@Haroenv Haroenv merged commit 6c18a10 into master Sep 30, 2020
@Haroenv Haroenv deleted the fix/vue-props-ssr branch September 30, 2020 08:11
Haroenv added a commit to algolia/instantsearch that referenced this pull request Dec 28, 2022
…antsearch#865)

* feat(ssr): forward propsData to recreated component

TODO: this throws a warning (but works) because of:

https://github.com/vuejs/vue/blob/7912f75c5eb09e0aef3e4bfd8a3bb78cad7540d7/src/core/util/options.js#L34-L44

I'm not fully sure how to do `new Vue` or pass this parameter correctly. If I `Object.assign` it for `app = new Vue({...extended, propsData})`, it also doesn't work :/

* fix

co-authored-by: Eunjae Lee <eunjae-lee@users.noreply.github.com>

Co-authored-by: Eunjae Lee <eunjae-lee@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants