-
Notifications
You must be signed in to change notification settings - Fork 513
[WIP] Implement Social Sign-In #2435
base: master
Are you sure you want to change the base?
[WIP] Implement Social Sign-In #2435
Conversation
Update So this code actually compiles and runs now, but I want to do some better error handling before asking for a review, and considering pushing this to production. This does contain some breaking changes, the most prominent one being the changes to the The If you try to run this locally, make sure you re-populate your |
Hey @gideonthomas I think this is ready for review, I've taken care of all the items I've outlined above, let me know if you've got any suggested changes, or need help getting this running. This isn't a very easy PR to review, it'll probably take a few hours just a heads up. |
Warning: This is still is progress, don't merge!