/names

Allow the plus sign in user names. #52

Merged
merged 2 commits into from Jul 9, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@frankban @jujubot
@frankban
Member

frankban commented Jul 9, 2015

So that the validation rules for users are consistent with what allowed by Launchpad.

(Review request: http://reviews.vapour.ws/r/2132/)
@frankban
Member

frankban commented Jul 9, 2015

Thanks for the reviews!
$$merge$$
@jujubot
Contributor

jujubot commented Jul 9, 2015

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju-names

jujubot added a commit that referenced this pull request Jul 9, 2015

@jujubot
Merge pull request #52 from frankban/allow-plus 
Allow the plus sign in user names.

So that the validation rules for users are consistent with what allowed by Launchpad.

(Review request: http://reviews.vapour.ws/r/2132/)
a6a253b

@jujubot jujubot merged commit a6a253b into juju:master Jul 9, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment