-
Notifications
You must be signed in to change notification settings - Fork 973
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
Fixes to crwa template for prerender #1819
Conversation
dac09
commented
Feb 19, 2021
- Fix entry js, not sure when rootElement went missing!
- Make index a little more readable, as it confuses people with the export default function
@peterp we broke something during our review 😅 |
Merging this peter, to unblock testing. We can discuss when you're back online! :) |
Whoops, I made these changes, do we need rootElement? I can't really understand how these changes break things? |
@peterp See lines 14 and 16? It's used there |
(Had this been TS it wouldn't even have compiled, just sayin... 😉) |
*/ | ||
if (document.getElementById('redwood-app').hasChildNodes()) { | ||
*/ | ||
const rootElement = document.getElementById('redwood-app') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, I was wondering about that!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the tip @thedavidprice. They don't call you el David for no reason!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lulz
I kinda want that on a hat
Could not agree with you more |
…rib-workspaces * 'main' of github.com:redwoodjs/redwood: Lint the create-redwood-app template. (redwoodjs#1822) Update typescript lint rule for no-unused-vars (redwoodjs#1808) Router: Fix TS types (redwoodjs#1823) Generate globals for routes (redwoodjs#1744) Ethereum auth update to v0.2.1 (redwoodjs#1807) Better messages when no routes marked with prerender (redwoodjs#1821) Feature/prerender image support (redwoodjs#1721) Fix entry js | Make index a little more readable, as it confuses people (redwoodjs#1819) use 'prisma' in place of '@prisma/cli' (redwoodjs#1800)