Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Enabled patient search by phone number #409

Merged
merged 1 commit into from
May 11, 2016
Merged

Enabled patient search by phone number #409

merged 1 commit into from
May 11, 2016

Conversation

iamquechua
Copy link
Contributor

@iamquechua iamquechua commented Apr 8, 2016

Changes proposed in this pull request:

  • Enabled patient search through their phone number.

Currently patient can only be searched with their id, firstname and lastname

cc @HospitalRun/core-maintainers

@jglovier
Copy link
Member

jglovier commented Apr 9, 2016

@Douno thanks for the pull request! To help us keep this project well documented, please complete the body of the pull request per the template provided (i.e. list out a summary of what you are proposing in the pull request, and in the Fixes # . line please reference any relevant issues that this pull address. Thanks!

@tangollama
Copy link
Member

@Douno I'd love to see this PR closed out by you completing the documentation process and correcting the CI errors in the branch. It's a worthwhile feature to add to the system. Would you have further time to invest in it?

@iamquechua
Copy link
Contributor Author

@tangollama I have updated the PR message. As for the CI errors, can you point me in the right direction. The change is so minor so minor so I don't understand why there are 11 fails. Cheers

@jkleinsc
Copy link
Member

@Douno thanks for the PR and the update! The CI errors were caused by an issue with our tests. I reran the tests and they now pass, so I am going to merge this PR.

@jkleinsc jkleinsc merged commit c10e068 into HospitalRun:master May 11, 2016
@jglovier
Copy link
Member

@Douno fantastic. Thanks for adding this! 🎉

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants