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

Updated landing page for redesign #154

Merged
merged 6 commits into from
Jul 30, 2024
Merged

Updated landing page for redesign #154

merged 6 commits into from
Jul 30, 2024

Conversation

ejmg
Copy link
Collaborator

@ejmg ejmg commented Jul 30, 2024

Closes #131

This PR updates the landing page to match those in the mocks. Added a PreviewTable component that provides a convenient wrapper for the un-paginated data-table ui components.

Caught an extremely obnoxious bug that only seems to exist in dev mode, on chrome, when not using --turbo as a feature, and when loading an initial page with a Suspense boundary on it (in this PR, HydrationBoundary). The update to the dev command should prevent this in the future.

This bug is a SyntaxError for the initial load of layout.tsx that only points to the minified webpack string. This eventually triggers a random hard runtime error that crashes the page because a chunk failed to load from .next after a minute or two.

From my own testing, it does not appear to emerge in any production builds but this PR is a convenient place to note the behavior in case the bug emerges again.

@ejmg ejmg merged commit 6ab549f into main Jul 30, 2024
@ejmg ejmg deleted the landing-page-update branch July 31, 2024 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Update landing (/) page
1 participant