Missing TS example of custom signin page

[peter-novak-m]

What is the improvement or update you wish to see?

Would it be possible to get the example of pages/auth/signin.js provided on this page:
https://next-auth.js.org/configuration/pages
in Typescript as well?

Is there any context that might help us understand?

I am working on my first full application using https://create.t3.gg/ and would like to create my own NextAuth signing page in a typesafe way, but found the documentation about this just to cover JS.

Does the docs page already exist? Please link to it.

No response

[shubhamku044]

Hey, can I take this issue? I will write the example in typescript as well.

[balazsorban44]

I would just prefer if we convert the code examples on https://next-auth.js.org/configuration/pages to TS

for simplicity, there's even an Edit this page button at the bottom.

[shubhamku044]

Hey @balazsorban44, I went through the next-auth/react folder. I found a few things

To convert, the code examples to TS we first have to create a proper type variable and then export the type variable. Please let me know if I am wrong.

[balazsorban44]

This is the only file needing an edit: https://github.com/nextauthjs/next-auth/edit/v4/docs/docs/configuration/pages.md

To be honest, @f98pno could you clarify exactly which block on that page caused issues for you with TypeScript? Most are very straightforward snippets.

[rawbinary]

isn't infering from getServerSideProps enough? the props we pass from there is already typed.

function SignIn({ providers }: InferGetServerSidePropsType<typeof getServerSideProps>)

Just adding the above shall be enough, I suppose and like @balazsorban44 said, everything else is pretty straightforward.

[balazsorban44]

Yeah, let's do that then, if OP does not comment on anything else, I'm inclined to close this issue with the above suggestion merged.

[rawbinary]

i guess, this issue is applicable for new docs as well (main branch) on https://authjs.dev/guides/basics/pages .
@balazsorban44 Should I create a PR for that as well, linking on this issue ?

[balazsorban44]

Not necessary, as the new docs are being rewritten from scratch. 👍 Thanks for the PR!