Skip to content
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

Add checks for real names in username #141

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

siddhpant
Copy link
Contributor

Closes #113

@siddhpant siddhpant force-pushed the recognize-real-names branch 2 times, most recently from 446cbf5 to 68b03f1 Compare November 11, 2018 13:15
real_name_checker.py Outdated Show resolved Hide resolved
static/css/main.css Outdated Show resolved Hide resolved
real_name_checker.py Outdated Show resolved Hide resolved
real_name_checker.py Outdated Show resolved Hide resolved
frontend.py Show resolved Hide resolved
real_name_checker.py Outdated Show resolved Hide resolved
@siddhpant siddhpant force-pushed the recognize-real-names branch 10 times, most recently from 65911e2 to 417c5e6 Compare November 12, 2018 16:52
templates/status.html Outdated Show resolved Hide resolved
templates/status.html Outdated Show resolved Hide resolved
templates/status.html Outdated Show resolved Hide resolved
templates/status.html Outdated Show resolved Hide resolved
test-requirements.txt Outdated Show resolved Hide resolved
real_name_checker.py Outdated Show resolved Hide resolved
frontend.py Show resolved Hide resolved
.travis.yml Outdated Show resolved Hide resolved
templates/status.html Outdated Show resolved Hide resolved
@siddhpant siddhpant force-pushed the recognize-real-names branch 3 times, most recently from 1206e76 to 2252b9c Compare November 13, 2018 05:57
.travis.yml Outdated Show resolved Hide resolved
@siddhpant siddhpant force-pushed the recognize-real-names branch 3 times, most recently from 21d968c to 88c3007 Compare November 13, 2018 08:13
Copy link

@SerekKiri SerekKiri left a comment

Choose a reason for hiding this comment

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

LGTM 👍

frontend.py Outdated Show resolved Hide resolved
templates/status.html Outdated Show resolved Hide resolved
templates/status.html Outdated Show resolved Hide resolved
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.

6 participants