{"action":"created","issue":{"url":"https://api.github.com/repos/thoughtbot/carnival/issues/227","labels_url":"https://api.github.com/repos/thoughtbot/carnival/issues/227/labels{/name}","comments_url":"https://api.github.com/repos/thoughtbot/carnival/issues/227/comments","events_url":"https://api.github.com/repos/thoughtbot/carnival/issues/227/events","html_url":"https://github.com/thoughtbot/carnival/pull/227","id":67558147,"number":227,"title":"Add unique constraint on UserEmail","user":{"login":"pbrisbin","id":50812,"avatar_url":"https://avatars.githubusercontent.com/u/50812?v=3","gravatar_id":"","url":"https://api.github.com/users/pbrisbin","html_url":"https://github.com/pbrisbin","followers_url":"https://api.github.com/users/pbrisbin/followers","following_url":"https://api.github.com/users/pbrisbin/following{/other_user}","gists_url":"https://api.github.com/users/pbrisbin/gists{/gist_id}","starred_url":"https://api.github.com/users/pbrisbin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pbrisbin/subscriptions","organizations_url":"https://api.github.com/users/pbrisbin/orgs","repos_url":"https://api.github.com/users/pbrisbin/repos","events_url":"https://api.github.com/users/pbrisbin/events{/privacy}","received_events_url":"https://api.github.com/users/pbrisbin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-04-10T10:00:57Z","updated_at":"2015-04-10T10:24:34Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/thoughtbot/carnival/pulls/227","html_url":"https://github.com/thoughtbot/carnival/pull/227","diff_url":"https://github.com/thoughtbot/carnival/pull/227.diff","patch_url":"https://github.com/thoughtbot/carnival/pull/227.patch"},"body":"- Re-use githubProfile for dummy logins because tests now need more\n realistic behavior regarding user emails\n- Replace upsertUser with separate replace/insert functions because\n upsert requires a single unique constraint on the entity"},"comment":{"url":"https://api.github.com/repos/thoughtbot/carnival/issues/comments/91507994","html_url":"https://github.com/thoughtbot/carnival/pull/227#issuecomment-91507994","issue_url":"https://api.github.com/repos/thoughtbot/carnival/issues/227","id":91507994,"user":{"login":"cpytel","id":5015,"avatar_url":"https://avatars.githubusercontent.com/u/5015?v=3","gravatar_id":"","url":"https://api.github.com/users/cpytel","html_url":"https://github.com/cpytel","followers_url":"https://api.github.com/users/cpytel/followers","following_url":"https://api.github.com/users/cpytel/following{/other_user}","gists_url":"https://api.github.com/users/cpytel/gists{/gist_id}","starred_url":"https://api.github.com/users/cpytel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cpytel/subscriptions","organizations_url":"https://api.github.com/users/cpytel/orgs","repos_url":"https://api.github.com/users/cpytel/repos","events_url":"https://api.github.com/users/cpytel/events{/privacy}","received_events_url":"https://api.github.com/users/cpytel/received_events","type":"User","site_admin":false},"created_at":"2015-04-10T10:24:34Z","updated_at":"2015-04-10T10:24:34Z","body":"How many users would that be?\n\nI feel like that's not a great option long term for what is likely a small number of people. We could even personally contact these people and help them out.\n\n\n\n> On Apr 10, 2015, at 6:14 AM, pat brisbin <notifications@github.com> wrote:\n> \n> We actually have a few real users with duplicate emails already because of the Learn->GitHub transition. I think this deploy should have a migration to:\n> \n> Find such users\n> Reassign comments from the learn user records to the github ones\n> Delete the duplicate learn users\n> One question is what happens to users who have commented after logging in via Learn, but not yet logged in via GitHub? When they attempt to log in next (assuming the same email), it'll fail with nothing they can do to correct the situation since we no longer have Learn as an option. I'm wondering if bringing back Learn (Upcase now) as an option is the only thing we can do.\n> \n> —\n> Reply to this email directly or view it on GitHub.\n> \n"}}