-
Notifications
You must be signed in to change notification settings - Fork 1
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
Chore/verify pages #167
base: main
Are you sure you want to change the base?
Chore/verify pages #167
Conversation
Hey there and thank you for opening this pull request! 👋🏼 We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted. Details:
|
176bf59
to
f1f1690
Compare
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.
Ok without the changes in apps/web/app/AppDirSSRHOC.tsx
7839cec
to
03c83dd
Compare
03c83dd
to
a7c80dd
Compare
This PR is being marked as stale due to inactivity. |
What does this PR do?
This PR migrates the /auth/verify pages to use the app directory (which runs under the App Router).
Migrated page exist under
/future/auth/verify
and is accessible at all times.Type of change
How should this be tested?
Pages performance can be accessed via their individual URLs and various metrics could be validated to be improved compared to the previous implementation.
Mandatory Tasks
Checklist