Comments (3)
Hey @sahat with Drywall we're not doing any account merging if a user creates multiple accounts, that sounds like complicated business. However, in our Account > Settings
area we do let people link their social accounts to their existing account.
from hackathon-starter.
IMO the first email ought to take priority, e.g., it is not overwritten by the local strategy.
In your example above, the user will receive a flash saying they already made an account
. Instead of displaying [email protected]
, it'll output [email protected]
on profile.jade.
This will take some logic to detect the edge case, but I think it's the best solution. It's also more predictable IMO since it preserves the earliest email account ([email protected]
).
from hackathon-starter.
@jedireza Thanks Reza. That's what I ended up doing it as well. After hours of trying to get the merging process right, it sometimes wouldn't save OAuth provider Id for whatever reason. And then there are issues with which account gets what. Instead I went with a simpler approach - to display a notification saying "Either sign in through that account, or delete it and link it with this current account in the account management page".
from hackathon-starter.
Related Issues (20)
- Learn node
- Can we use nodemon by default? HOT 2
- Brave browser blocks the cookieconsent HOT 1
- Node 20 HOT 4
- mongoose connection HOT 3
- Add JWT Authentication Support
- Implement Passport Serialization and Deserialization HOT 5
- Implement support for Novu as a notification infrastructure HOT 1
- Will make navbar remain on top on scrolling HOT 5
- Uniform text colour under "API Examples" tab HOT 11
- Creating seperate database config file HOT 2
- Destroying the cookies after logout of user HOT 2
- Adding OAUTH 2.0 HOT 3
- Refactoring config/passport.js HOT 5
- Mongo Running, Connection Error
- Asked assistant HOT 1
- Migrate from namespaced javaScript to Modular javaScript
- Encountering an issue with the script execution in Windows environment. HOT 1
- Critical dependency error during install: formidable <3.2.4
- node app.js fails HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from hackathon-starter.