Skip to content

Conversation

@wachterjohannes
Copy link
Member

@wachterjohannes wachterjohannes commented Jul 4, 2018

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets none
Related issues/PRs none
License MIT

What's in this PR?

This PR integrates the HWIOAuthBundle to allow social-media logins via facebook, google and co.

To Do

  • Tests
  • Cleanup code
  • Add Documentation
  • Make this feature optional
  • Fix confirmation Url generation when registered routes correctly as type portal routes (reported in slack)
  • Rebase for 1.x

@wachterjohannes wachterjohannes force-pushed the feature/some-login branch 3 times, most recently from 080fb26 to 29ad156 Compare July 4, 2018 11:59
$user->setEnabled(true);
$user->setLocale('de'); // FIXME locale

$this->em->persist($contact);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$contact->setLastName($response->getLastName());
$user->setContact($contact);

$user->setUsername($this->generateRandomUsername($username, $service));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could be generated from first part of the email address + uniquepostfix?

@alexander-schranz alexander-schranz changed the base branch from develop to master February 11, 2019 12:52
@alexander-schranz alexander-schranz changed the base branch from master to develop February 11, 2019 12:53
@niklasnatter niklasnatter changed the base branch from develop to 2.x January 12, 2021 12:44
@alexander-schranz alexander-schranz marked this pull request as draft April 25, 2024 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants