-
Notifications
You must be signed in to change notification settings - Fork 15
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: migrate api routes from pages to app #1266
Conversation
82b817d
to
db34000
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.
I find it very relevant to migrate this code to the app router as it is core code.
Could not make it work, both AgentConnect and FranceConnect does not work in local. FranceConnect might be unrelated to your changes though |
I fixed it. When using NextResponse.redirect, full url must be used. |
move files 2 move files 3
chore: use folder with route.ts files fix: migrate back to pages router download/espace-agent/documents/ fix: separate types from routes files fix: use full url for redirect in app/api fix
97c6201
to
d22d79d
Compare
api/auth/agent-connect
,api/auth/france-connect
.Closes #1267
Next thing : connexion pages migration in another PR (this one is required before merging agent habilitation PR)