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.

Sign up for GitHub

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

Jump to bottom

RESTWS-629 - Add new resource PersonNameResource2_1 #298

Merged
merged 1 commit into from
Sep 4, 2017
Merged

RESTWS-629 - Add new resource PersonNameResource2_1 #298

merged 1 commit into from
Sep 4, 2017

Conversation

gayanW
Copy link
Member

@gayanW gayanW commented Sep 1, 2017

No description provided.

@mention-bot
Copy link

@gayanW, thanks for your PR! By analyzing the history of the files in this pull request, we identified @teleivo to be a potential reviewer.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 40.328% when pulling 332d175 on gayanW:RESTWS-629 into d23b7dd on openmrs:master.

@gayanW gayanW changed the title RESTWS-629 - Adds new resource PersonNameResource2_1 RESTWS-629 - Add new resource PersonNameResource2_1 Sep 2, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 40.353% when pulling 48ac26d on gayanW:RESTWS-629 into d23b7dd on openmrs:master.

@dkayiwa
Copy link
Member

dkayiwa commented Sep 2, 2017

Did you just forget to squash these commits into one?

@gayanW
RESTWS-629 - Add new resource PersonNameResource2_1
3d3f638 
* Add new resource PersonNameResource2_1
* Add test PersonNameController2_1Test
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 40.353% when pulling 3d3f638 on gayanW:RESTWS-629 into d23b7dd on openmrs:master.

@gayanW
Copy link
Member Author

gayanW commented Sep 3, 2017

Done.

@dkayiwa dkayiwa merged commit d47bf26 into openmrs:master Sep 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@gayanW @mention-bot @coveralls @dkayiwa