Skip to content

RESTWS-755: Create user with email property#53

Open
tenas97 wants to merge 1 commit intoopenmrs:masterfrom
tenas97:master
Open

RESTWS-755: Create user with email property#53
tenas97 wants to merge 1 commit intoopenmrs:masterfrom
tenas97:master

Conversation

@tenas97
Copy link
Copy Markdown

@tenas97 tenas97 commented Aug 22, 2019

Part of work for GSOC 2019 project - Openmrs Password Reset User Interface. Add User Email field in the user creation form and ensure it is a required field. Ensure that user email is saved properly when user is edited

…uired field. Ensure that user email is saved properly when user is edited
@tenas97 tenas97 changed the title Add User Email field in the user creation form and ensure it is a required field. Ensure that user email is saved properly when user is edited RESTWS-755: Create user with email property Aug 24, 2019
@HerbertYiga
Copy link
Copy Markdown

did you see the failing checks here?

@tenas97
Copy link
Copy Markdown
Author

tenas97 commented Aug 29, 2019

Yeah I did, but I couldn’t figure out why it failed, certainly don’t think it’s due to the code I added. I think it’s an issue with the module itself, as you can see the other recent pull request made to this module failed the same check similarly.

@dkayiwa
Copy link
Copy Markdown
Member

dkayiwa commented Dec 12, 2019

@tenas97 can you discard all the formatting changes?

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