search

christillman / encounterpro_os

EncounterPro-OS Electronic Health Record Software
GNU Affero General Public License v3.0
0 stars 0 forks source link

Bug: Clicking on a prior medication appearing on a patient's prior med list deletes it from the list. #73

Open MCIR opened 1 week ago

MCIR commented 1 week ago

Contact Details

No response

What happened?

Clicking on a prior medication appearing on a patient's prior med list deletes it from the list.

How did it happen?

From a patient's chart page, click the 'Prior Medications' box to open the prior medications page. On this page, select a medication from the list of the patient's prior medications. Clicking on the medication will immediately delete it from the list, even if the appointment is closed.

Expectations

On the prior medications page, clicking on a patient's prior medication should not remove it from the list, regardless of whether the appointment is open or closed

Urgency and Importance

Medium urgency

Version

7.2.1.4

Relevant log output

No response

Code of Conduct

christillman commented 1 week ago

Hi Ciru,

Then there will be no way to revise prior medications. How do you suggest to revise them?

MCIR commented 1 week ago

Hi Chris,

Hmmh. I hadn’t considered it from that angle, although I recall we agreed on this method.

This caught me off guard because I was trying to expand on a patient’s prior medications. When I clicked on it, it disappeared. Unfortunately, I couldn’t clearly recall which drug had been recorded, but I tried my best to remember and added it back. I am still not sure it was the right drug.

For a different user, this could easily lead to the unintended loss of crucial patient information (though it can be regained by obtaining that information from the patient). I need to give this more thought since the ability to revise prior medications is also important. For now, I agree to leave it as it is.

From: Christopher Tillman @.> Sent: Friday, 20 September 2024 23:21 To: christillman/encounterpro_os @.> Cc: Madeline Wanjiru @.>; Author @.> Subject: Re: [christillman/encounterpro_os] Bug: Clicking on a prior medication appearing on a patient's prior med list deletes it from the list. (Issue #73)

Hi Ciru,

Then there will be no way to revise prior medications. How do you suggest to revise them?

— Reply to this email directly, view it on GitHub https://github.com/christillman/encounterpro_os/issues/73#issuecomment-2364633303 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AJT6WKLALLJOZV6ON4KTWB3ZXSGTNAVCNFSM6AAAAABOSRDQRKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRUGYZTGMZQGM . You are receiving this because you authored the thread. https://github.com/notifications/beacon/AJT6WKPPU7JFLRN774W7ZEDZXSGTNA5CNFSM6AAAAABOSRDQRKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUM6FYNO.gif Message ID: @. @.> >

christillman commented 1 week ago

Oh, OK, so the use case that's not currently covered (what you expected) was to show the detail of the listed drug. Instead of removing it straight away, we could put up a two-item menu with Remove and Details items (and Cancel).

On Sun, Sep 22, 2024 at 2:36 AM Madeline Wanjiru @.***> wrote:

Hi Chris,

Hmmh. I hadn’t considered it from that angle, although I recall we agreed on this method.

This caught me off guard because I was trying to expand on a patient’s prior medications. When I clicked on it, it disappeared. Unfortunately, I couldn’t clearly recall which drug had been recorded, but I tried my best to remember and added it back. I am still not sure it was the right drug.

For a different user, this could easily lead to the unintended loss of crucial patient information (though it can be regained by obtaining that information from the patient). I need to give this more thought since the ability to revise prior medications is also important. For now, I agree to leave it as it is.

From: Christopher Tillman @.> Sent: Friday, 20 September 2024 23:21 To: christillman/encounterpro_os @.> Cc: Madeline Wanjiru @.>; Author @.> Subject: Re: [christillman/encounterpro_os] Bug: Clicking on a prior medication appearing on a patient's prior med list deletes it from the list. (Issue #73)

Hi Ciru,

Then there will be no way to revise prior medications. How do you suggest to revise them?

— Reply to this email directly, view it on GitHub < https://github.com/christillman/encounterpro_os/issues/73#issuecomment-2364633303> , or unsubscribe < https://github.com/notifications/unsubscribe-auth/AJT6WKLALLJOZV6ON4KTWB3ZXSGTNAVCNFSM6AAAAABOSRDQRKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRUGYZTGMZQGM> . You are receiving this because you authored the thread. < https://github.com/notifications/beacon/AJT6WKPPU7JFLRN774W7ZEDZXSGTNA5CNFSM6AAAAABOSRDQRKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUM6FYNO.gif> Message ID: @. @.> >

— Reply to this email directly, view it on GitHub https://github.com/christillman/encounterpro_os/issues/73#issuecomment-2365210593, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGEGTU2LFY2LLWDQKC7GRNLZXWAALAVCNFSM6AAAAABOSRDQRKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRVGIYTANJZGM . You are receiving this because you were assigned.Message ID: @.***>

-- Chris Tillman Developer

MCIR commented 1 week ago

so the use case that's not currently covered (what you expected) was to show the detail of the listed drug.

Actually no. I am not sure why I was looking for more detail, and we really should not be expecting more detail from this selection, since it is already clear that the listing is for drug name only, not name and strength. I just selected it probably out of a habit of always looking for more details. We can keep this open and see what response I get from users.

From: Christopher Tillman @.> Sent: Saturday, 21 September 2024 21:20 To: christillman/encounterpro_os @.> Cc: Madeline Wanjiru @.>; Author @.> Subject: Re: [christillman/encounterpro_os] Bug: Clicking on a prior medication appearing on a patient's prior med list deletes it from the list. (Issue #73)

Oh, OK, so the use case that's not currently covered (what you expected) was to show the detail of the listed drug. Instead of removing it straight away, we could put up a two-item menu with Remove and Details items (and Cancel).

On Sun, Sep 22, 2024 at 2:36 AM Madeline Wanjiru @. <mailto:@.> > wrote:

Hi Chris,

Hmmh. I hadn’t considered it from that angle, although I recall we agreed on this method.

This caught me off guard because I was trying to expand on a patient’s prior medications. When I clicked on it, it disappeared. Unfortunately, I couldn’t clearly recall which drug had been recorded, but I tried my best to remember and added it back. I am still not sure it was the right drug.

For a different user, this could easily lead to the unintended loss of crucial patient information (though it can be regained by obtaining that information from the patient). I need to give this more thought since the ability to revise prior medications is also important. For now, I agree to leave it as it is.

From: Christopher Tillman @. <mailto:@.> > Sent: Friday, 20 September 2024 23:21 To: christillman/encounterpro_os @. <mailto:@.> > Cc: Madeline Wanjiru @. <mailto:@.> >; Author @. <mailto:@.> > Subject: Re: [christillman/encounterpro_os] Bug: Clicking on a prior medication appearing on a patient's prior med list deletes it from the list. (Issue #73)

Hi Ciru,

Then there will be no way to revise prior medications. How do you suggest to revise them?

— Reply to this email directly, view it on GitHub < https://github.com/christillman/encounterpro_os/issues/73#issuecomment-2364633303> , or unsubscribe < https://github.com/notifications/unsubscribe-auth/AJT6WKLALLJOZV6ON4KTWB3ZXSGTNAVCNFSM6AAAAABOSRDQRKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRUGYZTGMZQGM> . You are receiving this because you authored the thread. < https://github.com/notifications/beacon/AJT6WKPPU7JFLRN774W7ZEDZXSGTNA5CNFSM6AAAAABOSRDQRKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUM6FYNO.gif> Message ID: @. <mailto:@.> @. <mailto:@.> > >

— Reply to this email directly, view it on GitHub https://github.com/christillman/encounterpro_os/issues/73#issuecomment-2365210593, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGEGTU2LFY2LLWDQKC7GRNLZXWAALAVCNFSM6AAAAABOSRDQRKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRVGIYTANJZGM . You are receiving this because you were assigned.Message ID: @. <mailto:@.> >

-- Chris Tillman Developer

— Reply to this email directly, view it on GitHub https://github.com/christillman/encounterpro_os/issues/73#issuecomment-2365293970 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AJT6WKNFIBR7MCEG5TZOZQLZXXBGFAVCNFSM6AAAAABOSRDQRKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRVGI4TGOJXGA . You are receiving this because you authored the thread. https://github.com/notifications/beacon/AJT6WKO3DAVME4FNXX6WKFLZXXBGFA5CNFSM6AAAAABOSRDQRKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUM7OCZE.gif Message ID: @. @.> >