-
Notifications
You must be signed in to change notification settings - Fork 34
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
support wallet logon #64
Conversation
@JoviDeCroock is attempting to deploy a commit to the Developerdao Team on Vercel. A member of the Team first needs to authorize it. |
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.
Awesome stuff @JoviDeCroock !
Just got a rename request for the attribute to store the wallet address and let's add nonce
Thanks for the reviews! I'll wait for the database changes to go live, then test and convert to "ready for review" |
Merged my PR and updated the database, you can test out the new changes. |
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.
Very awesome @JoviDeCroock ! 💯
@@ -17,19 +17,20 @@ | |||
"@usedapp/core": "^0.5.4", | |||
"babel-plugin-styled-components": "^1.13.3", | |||
"cross-env": "^7.0.3", | |||
"doppler": "^1.0.0", |
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.
Noticed this was unused and not present in the lockfile. Removed it as it's unused and a risk for versioning 😅
This is in anticipation of the new database schema, this allows us to logon using our
metamask
/... wallet. We send the address provided by the wallet up to/api/auth
and this will send us back whether or not we're valid and create/give us back an account from supabase.After that we can redirect our user to
/create-profile
when they're new or to/jobs
resolves #4