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

update template client entry to match remix #604

Merged
merged 2 commits into from
Mar 3, 2023

Conversation

lordofthecactus
Copy link
Contributor

@lordofthecactus lordofthecactus commented Mar 1, 2023

Enable time-slicing with React.startTransition to prevent hydration from blocking the main thread for too long for those users who immediately scroll.

Based on Remix templates

@lordofthecactus lordofthecactus requested a review from a team March 1, 2023 19:48
Copy link
Contributor

@juanpprieto juanpprieto left a comment

Choose a reason for hiding this comment

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

👍🏼

Copy link
Contributor

@cartogram cartogram left a comment

Choose a reason for hiding this comment

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

👍

We should set a plan on how best to version and test templates.

Copy link
Contributor

@frandiox frandiox left a comment

Choose a reason for hiding this comment

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

Since you are updating the entries, can you also update the entry.server.tsx with this? https://github.com/remix-run/remix/blob/a7661fc3228a0f34e9d9953938cc4bdbcaaf52b1/packages/remix-dev/config/defaults/entry.server.cloudflare.tsx

They've added better error handling there.
These files are now optional if we were deploying to Cloudflare or Deno. I don't think they can detect Oxygen yet, though.

@lordofthecactus
Copy link
Contributor Author

@frandiox thanks for the tip. I updated it as well. cc @cartogram @juanpprieto

@lordofthecactus lordofthecactus merged commit 4514286 into 2023-01 Mar 3, 2023
@lordofthecactus lordofthecactus deleted the update-templates-client branch March 3, 2023 10:11
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.

None yet

4 participants