Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

Multiple users can register with the same email address with default app #118

Closed
crewshin opened this issue Jul 19, 2014 · 10 comments
Closed
Assignees
Milestone

Comments

@crewshin
Copy link

While registering users with a default app, it's possible to register multiple users with the same email address. It does however check if the username exists and prompts the user to change.

@lirantal
Copy link
Member

anyone objects for also making the email address field unique?

@rschwabco
Copy link
Member

I have no objection - but we should implement a mechanism to resolve the use of multiple social accounts with the same email. IMO they can just be joined with the existing account info.

@crewshin
Copy link
Author

Fully agree with Roie.

Gene Crucean
www.genecrucean.com
@GeneCrucean <https://twitter.com/#!/GeneCrucean>

On Tue, Oct 28, 2014 at 12:17 PM, Roie Cohen <notifications@github.com>
wrote:

> I have no objection - but we should implement a mechanism to resolve the
> use of multiple social accounts with the same email. IMO they can just be
> joined with the existing account info.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/meanjs/mean/issues/118#issuecomment-60814783>.
>

@ilanbiala
Copy link
Member

@crewshin identities and providers are being discussed in #337. You may want to subscribe to that issue and chip in with your 2c.

@SOSANA
Copy link

SOSANA commented Mar 28, 2015

Any updates?

@ilanbiala
Copy link
Member

@SOSANA the plan is to handle that through #343, which needs to be merged into 0.4.0 rather than master. @sielay needs to create a new PR for 0.4.0 to avoid having lots of merge issues later on.

@codydaig
Copy link
Member

Since this moved to #343, and #343 was closed, can we close this issue?

@lirantal
Copy link
Member

@codydaig #343 is closed until we get that PR against 0.4.0 so maybe we leave this thread open still.

@codydaig
Copy link
Member

I believe this is taken care of now in #439

@lirantal
Copy link
Member

right, I'll close this @crewshin unless you spot any other problems with the fix in the latest 0.4.0 branch then you're welcome to re-open.

@lirantal lirantal self-assigned this Jul 24, 2015
@lirantal lirantal added this to the 0.4.0 milestone Jul 24, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants