Skip to content

Conversation

lightalloy
Copy link
Contributor

What type of PR is this? (check all applicable)

  • Optimization

Description

  • validate github_username and twitter_username uniqueness only if the corresponding field was changed to lower amount of requests when signing in and in other cases
  • change allow_blank to allow_nil options in these fields' validations, since only nil is allowed to be written to the database.

Related Tickets & Documents

#2061

@pr-triage pr-triage bot added the PR: unreviewed bot applied label for PR's with no review label Apr 5, 2019
Copy link
Contributor

@benhalpern benhalpern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@pr-triage pr-triage bot removed the PR: unreviewed bot applied label for PR's with no review label Apr 5, 2019
@benhalpern benhalpern merged commit 1b9f243 into forem:master Apr 5, 2019
@pr-triage pr-triage bot added PR: reviewed-approved bot applied label for PR's where reviewer approves changes PR: merged bot applied label for PR's that are merged labels Apr 5, 2019
@lightalloy lightalloy deleted the usernames-conditional-validation branch March 9, 2020 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: merged bot applied label for PR's that are merged PR: reviewed-approved bot applied label for PR's where reviewer approves changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants