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

Improve User retrieval. #19

Open
richardkmichael opened this issue Sep 2, 2011 · 4 comments
Open

Improve User retrieval. #19

richardkmichael opened this issue Sep 2, 2011 · 4 comments

Comments

@richardkmichael
Copy link
Owner

We look up a User based on their GitHub email address. If their address at GitHub changes, we can't look them up.

app/models/user.rb

  User.find_by_email(data['email'])

Ideas:

  • Just print a warning? "If you have changed your email address on GH ... "
  • Use the GH user id instead? (Could it change too?)
@franckverrot
Copy link
Collaborator

The user id can't change on GH, this is probably the only unique thing that can't change (the username can change, once only).

@richardkmichael
Copy link
Owner Author

I'd still need some absolute beer-app UI ability to update this info probably, as a total failsafe .. In case someone on GH creates a new account (GH UID change) and says "this is now me".

But it's a small corner-case. I was indeed thinking about using the GH UID.

@franckverrot
Copy link
Collaborator

This is so not supposed to happen you should consider asking people to open a ticket in your very own ticketing system and fix this manually. I would do so.

@richardkmichael
Copy link
Owner Author

Yes, perhaps a good tradeoff of time/money to implement a solution for a tiny corner-case. Just fix it by hand as a support ticket untils it's a common enough problem.

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

No branches or pull requests

2 participants