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

TRUNK-4294 [Add User] Password requirements should be shown beforehand as per the global properties #769

Merged
merged 1 commit into from Mar 12, 2014

Conversation

ujjwalarora
Copy link
Member

@ujjwalarora ujjwalarora mentioned this pull request Mar 10, 2014
@@ -1884,7 +1886,6 @@ error.username.taken=Username or System Id taken. Please choose a different one.
error.username.pattern=Username is invalid. It must be between 2 and 50 characters. Only letters, digits, ".", "-", and "_" are allowed.
User.password=Password
User.usersPassword=User's Password
User.password.description=Must have at least 8 characters, an uppercase letter, and a number
Copy link
Member Author

Choose a reason for hiding this comment

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

Added earlier in https://github.com/openmrs/openmrs-core/pull/721/files
Deleted because it doesn't need to be hard-coded. I didn't use global properties previously.

rkorytkowski added a commit that referenced this pull request Mar 12, 2014
TRUNK-4294  [Add User] Password requirements should be shown beforehand as per the global properties
@rkorytkowski rkorytkowski merged commit f338537 into openmrs:master Mar 12, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants