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-3010 - Clarify patient dashboard error ServletException: There is no patient with id: xxx #679

Merged
merged 2 commits into from Mar 10, 2014
Merged

Conversation

jasper-vandemalle
Copy link
Contributor

This is a fix for issue https://tickets.openmrs.org/browse/TRUNK-3010.
The error message will be handled properly now if the user opens an url with non-existing patient id.

@ghost
Copy link

ghost commented Mar 6, 2014

Hi @jasper-vandemalle. In future could you pllease kindly use a more a descriptive pull request title, similar to that as described for commit messages in https://wiki.openmrs.org/display/docs/Using+Git under step 3 in section "Work on your task". The comment was helpful but a reviewer might not immediately see it. Thanks!

@jasper-vandemalle
Copy link
Contributor Author

@downeym: I've modified the PR title. It now matches the title of the ticket.

dkayiwa added a commit that referenced this pull request Mar 10, 2014
TRUNK-3010 - Clarify patient dashboard error ServletException: There is no patient with id: xxx
@dkayiwa dkayiwa merged commit a432591 into openmrs:master Mar 10, 2014
@jasper-vandemalle jasper-vandemalle deleted the TRUNK-3010 branch March 10, 2014 12:19
RandilaP pushed a commit to RandilaP/openmrs-core that referenced this pull request Jul 31, 2023
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