-
Notifications
You must be signed in to change notification settings - Fork 479
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename primary_authentication_option to primary_contact_info #23387
Conversation
…o_id Also updates the association on User, but then introduces alias getter and setter methods for the old name to avoid breaking existing code. I'm going to run tests against this commit on its own first.
The first commit, with the aliased accessors for the old name, passed on Circle. I feel pretty good about that approach now. Pushing additional changes... |
dashboard/app/models/user.rb
Outdated
# TEMPORARY: Remove these aliases | ||
# | ||
|
||
def primary_authentication_option |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❤️ these aliases
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note, this can be much more condensed with alias_attribute
alias_attribute :primary_authentication_option, :primary_contact_info
…hentication-option-2
Discussing potential impact of the column rename on the users table (in particular, odds of causing downtime) in Slack: https://codedotorg.slack.com/archives/C03CK49G9/p1530202891000427 |
A second take on #23231 after we made lots and lots of changes in staging out from under that PR.
The first change is that I've started fresh from a more recent point on
staging
so I'm updating more of our recent work. The second change is that I've leaving in aliases for the old column/association name, to avoid breaking other PRs we have open. Then later we can do the fairly safe change of finishing the rename and removing those alias methods.