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

Server-side rendering fix #88

Merged
merged 1 commit into from
Nov 19, 2016
Merged

Server-side rendering fix #88

merged 1 commit into from
Nov 19, 2016

Conversation

parallelo3301
Copy link
Contributor

@parallelo3301 parallelo3301 commented Nov 18, 2016

Hello,
I tried to fix problems with server-side rendering.

The problem is in vm._data.__ob__, which becomes undefined when SSR occurs. I have { langs: { en: { message: "translation"}}} in vm._data, so this is fixed by checking for __ob__ existence.

Using of Dep is fixed by this.

Didn't study the behavior of vue-i18n well, but it works (with vue-hackernews-2.0) and SSR.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 91.375% when pulling 9b894f4 on fandaa:fix/ssr into 2bfe9fd on kazupon:dev.

@kazupon kazupon merged commit 724974e into kazupon:dev Nov 19, 2016
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.

3 participants