Conversation
jarednorman
left a comment
There was a problem hiding this comment.
What was the purpose of this column originally?
|
Maybe it's not related to the same thing, but there was a Considering this comment, is it possible to consider it related to unfinished and/or delayed plans? |
|
If there's not documentation as to what those plans were and the functionality doesn't exist now, I don't think it's worth keeping. |
|
I was about to submit this PR just now, and second the removal of this column. It stores a user's email address in @willianveiga would you be up to rebasing it? Otherwise I'll put up another PR with your changes here within a week. |
|
@mamhoff did you finish this up in the end? |
|
Devise can be configured to which authentication keys it uses. If this is configured to use login (speak username) instead of email, we need this column. But this is personal preference of each store and maybe should be added there instead. |
|
So to summarise:
And close PRs / tickets related to this topic. |
Remove unused login column.
Fixes #116