Skip to content
This repository has been archived by the owner on Jan 20, 2024. It is now read-only.

feat: added github sign in #282

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

abdelopx
Copy link

@abdelopx abdelopx commented Oct 8, 2022

🛠️ Fixes Issue

Closes #85

➕ Changes Introduced

  • Uncomment code responsible for showing the GitHub sign in logo (pages/singin.tsx)
  • Added support for GitHub sign in using Supabase (components/SignIn/Social.tsx)

📄 Note To Reviewers

📷 Screenshots

@vercel
Copy link

vercel bot commented Oct 8, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
defaang ✅ Ready (Inspect) Visit Preview Oct 8, 2022 at 9:37PM (UTC)

@abdelopx
Copy link
Author

abdelopx commented Oct 9, 2022

@ykdojo what do you think ?

@ykdojo
Copy link
Collaborator

ykdojo commented Oct 10, 2022

Thank you, but I don't think should be our priority right now. Happy to revisit it at some point in the future, though.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] Add multiple ways for authentication
2 participants