astriaai / headshots-starter

https://headshots-starter.vercel.app
MIT License
3.56k stars 575 forks source link

Update README.md with Confirm signup info #101

Open akdeb opened 4 months ago

akdeb commented 4 months ago

This should enable Confirm signup to work for new users

Proposed README changes:

For signup:

To make sure sign up works through the supabase "Confirm signup" tab. Add "/auth/callback" after {{ .ConfirmationURL }} like href="{{ .ConfirmationURL }}/auth/callback" so the project catches the sign up route in the right callback.
vercel[bot] commented 4 months ago

@akdeb is attempting to deploy a commit to the Leap AI Team on Vercel.

A member of the Team first needs to authorize it.

akdeb commented 4 months ago

This is a temp fix until this issue is fixed https://github.com/supabase/auth-helpers/issues/716#issue-2066349576 Ideally when emailRedirectTo is working correctly, we want to set this to

emailRedirectTo: `${requestUrl.origin}/auth/callback/`
agnelnieves commented 3 months ago

Hey, is this still an issue? I am having the same issue where i get the 401: invalid claim: missing sub claim error