Attempt to fix pre-render errors #137
Conversation
|
What are pre-render errors? Do we have an issue for this? Did you test this on your fork? Does it solve the issue? |
|
I should've been a bit more precise regarding where the issue stems from, sorry. TL;DR There's nothing to do with this PR. It should be fixed by #138. Let's discuss if needed.
The CI fails on master for some reason: "Pre-render" is a term which is frequently used in the case of SSGs (it's one of their fundamental reasons why they're so popular). What it basically refers to is generating each page way ahead of time (pages are compiled to static HTML) which results in 1) decreased loading time (because the app can be served from the "edge", be cached etc) 2) better user-experience (there's something to look at while the page is loading). One thing that's important to note here is that even though the pages are static, they're hydrated to become a fully interactive application if needed (e.g. fetch data at client-side when a component mounts). Resources if you're interested in this topic:
Right, it's my bad. I should've tested the changes against my forked repo instead of "guessing". Turns out, the changes do not fix the error whatsoever: Line 11 in d051894 As we pointed out the other day, the |
|
Closed because of #137 (comment). |
I'm just guessing because the error itself is somewhat vague. Tried to follow their recommendations to find out what the root cause is but to no avail (it's an error that cannot be reproduced locally).
The issue definitely has something to do with: https://github.com/c-hive/dotdev/pull/131/files
My gut feeling is that there's something wrong with that "_app" component though it's identical to their example. Let's remove this and inject the
<Head />component in the root path (mostly because it's a single-page app without different routes with different titles).