-
Notifications
You must be signed in to change notification settings - Fork 27
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
Web - Facebook Sign In #7
base: main
Are you sure you want to change the base?
Conversation
…ive-firebase.d.ts)
# Conflicts: # template/src/app/auth-providers/Google.tsx
# Conflicts: # template/src/app/auth-providers/Google.tsx
# Conflicts: # template/src/app/auth-providers/Google.tsx # template/src/app/signed-in/Settings.tsx # template/src/shims/firebase-auth-web.ts
# Conflicts: # template/src/app/auth-providers/Facebook.tsx # template/src/app/signed-in/Settings.tsx
FWIW - a rebase merge is a much much cleaner way to do development as it leaves a clean commit trail, this one is a real hash now - there are 18 commits for one feature 🤔 - I'm still on holidays though and have this queued for as soon as I return when I can start cleaning up my todo list for the new year :) |
Oh sorry about that, will keep that in mind. There are so many commits because I have the changes with the Google Sign in as well. The login on web seems to work well, but other than logging, there's no support yet. I'm on holidays as well, though I have to admit, those are the worst holidays in my life, but working with React Native is a good anti-depressant for me 😂 |
This PR contains the changes from on #6 as well,
It adds limited Facebook and Google functionality on the web using Firebase. But for demonstration purposes, I think it's sufficient.