-
Notifications
You must be signed in to change notification settings - Fork 51
(feat) O3-4998 Add drug dispense label print function #464
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
base: main
Are you sure you want to change the base?
(feat) O3-4998 Add drug dispense label print function #464
Conversation
|
Hi @PatrickWaweru, can you give us a brief description of this new feature? How is different from the existing "Print Prescription" functionality? |
Hi @chibongho This is the label that is normally stuck to the dispensing package or bottle. It might not contain the same details as the prescription. For example: The label is different from the prescription in these areas:
|
|
@PatrickWaweru this PR has changes to 96 files... I suspect it's outdated and needs to be updated with the latest dispensing code so we can properly review? |
5ec2566 to
e14c73f
Compare
Hi @mogoodrich This is now fixed |
chibongho
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like we are duplicating a lot of similar code from printing Prescription to get the printing Label to work. We can try to reuse code more. For example, instead of:
<PrescriptionPrintAction encounterUuid={encounterUuid} patientUuid={patientUuid} />
<LabelPrintAction encounterUuid={encounterUuid} />
we can do:
<PrintAction encounterUuid={encounterUuid} patientUuid={patientUuid} printMode="prescription"/>
<PrintAction encounterUuid={encounterUuid} patientUuid={patientUuid} printMode="label"/>
| </span>{' '} | ||
| —{' '} | ||
| <span className={styles.prescriptionInfo}> | ||
| {dosageInstruction?.route?.text} — {dosageInstruction?.timing?.code?.text} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This entire string <code> for <duration> <unit> needs to be translated as one string, as we cannot assume that those things will appear in that order in a different language. See this for example on how to add template variables within the translation string.
| <span className={styles.faintText}>{t('quantity', 'Quantity')}</span> | ||
| {': '} | ||
| <span className={styles.prescriptionInfo}> | ||
| {quantity.value} {quantity.unit} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here. Needs to be a translation
| )} | ||
| </p> | ||
| <p> | ||
| <span className={styles.faintText}>{t('datePrescribed', 'Date prescribed')}</span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the ':' needs to be part of the translation string (especially for rtl languages)
Requirements
O3-4998 Add functionality to print a drug label during drug dispensing.
This is the label that is normally stuck to the dispensing package or bottle. It might not contain the same details as the prescription. For example: The label is different from the prescription in these areas:
e.g., “May cause drowsiness,” “Keep refrigerated,” “Finish the full course”
Summary
Add a button to print a drug label during drug dispensing
Screenshots
Drug-Label.webm
Related Issue
Other