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

Fix Fast Refresh for WithPageAuthRequired #653

Merged
merged 1 commit into from
Apr 29, 2022
Merged

Conversation

adamjmcgrath
Copy link
Contributor

Description

After a bit of trial and error, I discovered that this function needs to be named as Title Case in order to work with Fast Refresh.

References

fixes #652

@adamjmcgrath adamjmcgrath added the review:tiny Tiny review label Apr 29, 2022
@adamjmcgrath adamjmcgrath requested a review from a team as a code owner April 29, 2022 11:55
@vercel
Copy link

vercel bot commented Apr 29, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated
nextjs-auth0 ⬜️ Ignored (Inspect) Apr 29, 2022 at 11:55AM (UTC)

@Widcket Widcket merged commit 1dcab73 into main Apr 29, 2022
@Widcket Widcket deleted the adamjmcgrath-patch-4 branch April 29, 2022 16:13
@adamjmcgrath adamjmcgrath mentioned this pull request May 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review:tiny Tiny review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

withPageAuthRequired breaks hot reloading
2 participants