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

Always remove extract-css-loader on server #191

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bogdantica
Copy link

... even when cricitcalCSS is disabled. This fixes the document is not defined bug in production mode
Fixes #186

phil294 and others added 2 commits March 23, 2020 13:25
... even when `cricitcalCSS` is disabled. This fixes the `document is not defined` bug in production mode
Fixes Akryum#186
@sneko
Copy link

sneko commented Jun 28, 2020

Interested to have this merged 👍

@sneko
Copy link

sneko commented Jul 8, 2020

Just decided to use my own fork with v0.6.0.

I did it be aliasing this repo:

yarn add "@akryum/vue-cli-plugin-ssr@https://github.com/sneko/vue-cli-plugin-ssr#fix/disabling-criticalcss"

I really hope in the future all critical PR will be merged, it's a great SSR plugin but it comes with too many breaks 😢

sneko added a commit to sneko/vue-cli-plugin-ssr that referenced this pull request Jun 10, 2022
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.

"document is not defined" when criticalCSS is disabled in production
3 participants