Skip to content
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

[FEAT] Login by social providers (Google for now) #1326

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

giovaborgogno
Copy link

@giovaborgogno giovaborgogno commented May 9, 2024

Pull Request Type

  • ✨ feat
  • πŸ› fix
  • ♻️ refactor
  • πŸ’„ style
  • πŸ”¨ chore
  • πŸ“ docs

Relevant Issues

resolves #1193

What is in this change?

This PR introduces the ability for users to log in using their Google accounts, providing the flexibility for administrators to enable or disable this feature. Additionally, administrators can restrict login access to a specific organization's domain. The implementation is designed with a modular structure, allowing for easy integration of additional authentication providers in the future.

Additional Information

image

After enable login by google:

image

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

@giovaborgogno giovaborgogno marked this pull request as ready for review May 10, 2024 00:02
@timothycarambat
Copy link
Member

Damn, this is awesome!

@emilioh93
Copy link

Hey @timothycarambat, are there any intentions to work on this PR? It looks really good

@timothycarambat
Copy link
Member

@emilioh93 There is nothing wrong with the PR, it just needs to be reviewed and we need documentation to set it up because I dont want to be responsible for 400 github issues from people setting it up incorrectly and saying it is broken!

From what i already tested, it works great & will be merged

@emilioh93
Copy link

emilioh93 commented May 15, 2024

@emilioh93 There is nothing wrong with the PR, it just needs to be reviewed and we need documentation to set it up because I dont want to be responsible for 400 github issues from people setting it up incorrectly and saying it is broken!

From what i already tested, it works great & will be merged

@timothycarambat, thanks for your reply. I agree that documentation is crucial to avoid configuration issues. I appreciate your time and effort in reviewing this PR.

@timothycarambat timothycarambat removed their assignment May 24, 2024
@matejc
Copy link

matejc commented Jun 5, 2024

What are the places where should one put the documentation?

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

Successfully merging this pull request may close these issues.

Implement Single Sign-On (SSO) Authentication with Azure Active Directory, GitHub, and Google
4 participants