(fix) Expand clearSearchResults label to avoid ambiguity with test results#3169
(fix) Expand clearSearchResults label to avoid ambiguity with test results#3169Skengmann wants to merge 1 commit into
Conversation
…sults The clearSearchResults key's value was truncated to "Clear results", which is ambiguous in the orders context where "results" also refers to lab/test results. Expanding it to "Clear search results" matches the key's intent and removes the ambiguity.
|
Hi @Skengmann ,Please include a ticket number in the pull request title, description and commit message . This is required as per the OpenMRS developer guidelines. Kindly review the guidelines and update the PR accordingly. Thanks! |
|
Thank you for the feedback. I attempted to create Jira tickets but am
having trouble accessing the OpenMRS Atlassian instance. These are minor
translation consistency fixes with no associated feature work. I am happy
to create tickets if you can point me to the correct process. I am a GSoC
2026 applicant and this is my first contribution to the codebase.
…On Wed, 25 Mar 2026 at 15:10, Pratik Debnath ***@***.***> wrote:
*Pratikdeb07* left a comment (openmrs/openmrs-esm-patient-chart#3169)
<#3169 (comment)>
Hi @Skengmann <https://github.com/Skengmann> ,Please include a ticket
number in the pull request title, description and commit message . This is
required as per the OpenMRS developer guidelines. Kindly review the
guidelines and update the PR accordingly. Thanks!
—
Reply to this email directly, view it on GitHub
<#3169?email_source=notifications&email_token=A2SBYF22HYGMEXUKNQ7BYVL4SPSD5A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIMJSGY4TAOBXGAZKM4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#issuecomment-4126908702>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A2SBYFZLEOVY4LW3N4H4XZT4SPSD5AVCNFSM6AAAAACW63TR32VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DCMRWHEYDQNZQGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
Good eye on the ambiguity — "Clear results" could definitely be confused with lab/test results in the orders context. Same as on #3170 — Also a small thing — the PR description has unclosed HTML comment tags ( |
Requirements
Summary