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

Profanity check #577

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Vedantdesai2511
Copy link

Added function to check profanity in the user name and throw an error if a bad word is found in the user name
The function is implemented to check profanity in the change_username (to change user name) function and create_user (to make a new user) function

An example error is given below

image

image

Added fucntion to check profanity in user name
Added profanity check for new user name
profanity check error message grammer capitalization fix
Copy link
Collaborator

@mliu59 mliu59 left a comment

Choose a reason for hiding this comment

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

LGTM!

@dqian3
Copy link
Collaborator

dqian3 commented Jan 28, 2024

Tests are failing?

@dqian3
Copy link
Collaborator

dqian3 commented Jan 28, 2024

Ahh, please add profanity_check to requirements.txt, that should fix it.

@mliu59
Copy link
Collaborator

mliu59 commented Feb 24, 2024

found the right package version. @dqian3 pls merge when ready

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.

3 participants