Skip to content

suggest username when taken#4

Open
pebble-fish wants to merge 3 commits intoCMU-313:mainfrom
pebble-fish:username-suggestion
Open

suggest username when taken#4
pebble-fish wants to merge 3 commits intoCMU-313:mainfrom
pebble-fish:username-suggestion

Conversation

@pebble-fish
Copy link
Copy Markdown

Closes #1

What this PR does

suggests an alternative username when the desired username is taken by appending values to the end of their chosen username.

Changes

  • detects whether the username is taken and suggests a new username by appending a numeric suffix
  • continuously suggests usernames until either 20 suggestions or a username that hasn't been taken has been found

How to test

  1. register with an existing username
  2. observe the suggested alternative username

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 21597738311

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 78.841%

Totals Coverage Status
Change from base Build 21503398961: 0.0%
Covered Lines: 25358
Relevant Lines: 30328

💛 - Coveralls

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Suggest a new username if a username is taken during registration

2 participants