Messages to multiple patients #1132

shachar058 · 2017-10-17T10:20:13Z

Hi Brady,
This addition enables the user to send a message to multiple patients (and acts pretty much like sending to multiple users which is an existing feature).
Everything is supposed to work the same besides for being able to send to multiple patients.

thanks,
shachar

(below added by bradymiller)
demo is here:
https://www.open-emr.org/wiki/index.php/Development_Demo#Lambda_-_Up_For_Grabs_Demo

bradymiller · 2017-10-18T19:10:37Z

This feature can be tested on the following Up For Grabs demo:
https://www.open-emr.org/wiki/index.php/Development_Demo#Lambda_-_Up_For_Grabs_Demo

shachar058 · 2017-10-19T08:14:59Z

@bradymiller
I'm not sure what you mean...
Do you want me to test it on that environment?
When I go to the link https://four.openemr.io/h/openemr it gives me 502 bad gateway. Is this the link I'm meant to use?

bradymiller · 2017-10-19T08:25:22Z

Nope, the Up For Grab demos are just meant for other folks that want to test out this feature(without needing to install it locally).
Demos all reset at 1:00 pacific time and takes about 15 minutes which is why it didn't work for you. It's now working.

bradymiller · 2017-10-21T07:32:33Z

interface/main/messages/messages.php

- $reply_to = $_POST['reply_to'];
+
+ //Need to do this because sometimes there will be multiple patients separated by semicolon, sometimes 1 patient with semicolon, and sometimes 1 patient without semicolon
+ if (strpos($_POST['reply_to'], ';') !== false){


Isn't this only supposed to be true if the semicolon is at the end?

true, changed it to rtrim

bradymiller · 2017-10-21T07:37:28Z

@shachar058 ,
How is your group using patient messaging? Are you using the offsite patient portal or a custom module?
-brady

shachar058 · 2017-10-22T08:33:10Z

@bradymiller
Currently we aren't using messaging to patients (just between users).

bradymiller · 2017-10-22T08:36:46Z

@shachar058 ,
Then how are you testing this? For example, I am pretty sure this would break patient messaging in the offsite portal.
-brady

shachar058 · 2017-10-22T08:39:47Z

@bradymiller
We don't use it, but I have turned on offsite patient portal messaging for my testing. I didn't encounter any problems so far, but correct me if I missed something.
Why do you think it would break?

bradymiller · 2017-10-23T20:08:29Z

@shachar058 ,
Did it work on the offsite patient portal (ie. the ZH portal)? Just guessing it won't work since this was not the behavior previously. Just let me know how the testing goes. If it doesn't work, then will make sense to touch base with ZH since they'd likely want to add support for this feature.
-brady

shachar058 · 2017-10-24T08:49:03Z

@bradymiller
I've never used ZH portal, so not even sure how to access it.
Could you maybe expand on this?

bradymiller · 2017-10-25T08:25:05Z

@shachar058 ,
Here's instructions for using the offsite portal:
https://www.open-emr.org/wiki/index.php/Patient_Portal#Instructions_3
(note that this is the only feature that uses this mechanism for patient messaging)

@sjpadgett , just to confirm, your patient portal does not use pnotes for patient messaging, correct?

shachar058 · 2017-10-25T10:52:05Z

@bradymiller
I followed the instructions for the patient offsite portal and all went well until I tried logging in to the mydocsportal, which responded with 'Invalid Access'.
Tried a few times with same problem, not sure why as I'm entering the creds exactly as told in the instructions.
Basically, having a lot of trouble with getting this ZH portal to work...

sjpadgett · 2017-10-25T13:57:59Z

I use pnotes to mainly forward secure messages from a patient to another user from Dashboard. I also use it in Postings to send reminder to provider that an invoice has been sent to patient.

shachar058 · 2017-10-29T08:36:24Z

Hi @bradymiller,
I tried using the ZH portal again, but still am not able to access the portal.
If this testing is necessary then I'm a little stuck here, not sure how to proceed with this...

bradymiller · 2017-10-30T16:18:37Z

hi @shachar058 ,
Let me see if I can get the portal going tonight on my local instance. If issues, then will bring in ZH to see if can help.
-brady

sjpadgett · 2017-10-30T16:20:57Z

Would this be useful in my portal? Should I be looking at?

…

On Mon, Oct 30, 2017 at 12:18 PM, Brady Miller ***@***.***> wrote: hi @shachar058 <https://github.com/shachar058> , Let me see if I can get the portal going tonight on my local instance. If issues, then will bring in ZH to see if can helpl. -brady — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1132 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABxhkQ1EYAABOr3grfpuLklzqO2mnpe8ks5sxfbhgaJpZM4P76qS> .

-- Jerry Padgett Padgett's Consulting Brandon, Florida 33511 [email protected] <[email protected]>

bradymiller · 2017-10-30T16:23:03Z

I don't really know(since not sure how useful this is).
@shachar058 , what specific use case(s) is this feature(sending message to multiple patients) fulfilling?

shachar058 · 2017-10-31T08:17:29Z

Hi @bradymiller
In our clinic when the patients come in to get their medicine (once or twice a week for example), the doctor reads them the messages they have in their patient file. These can include reminders for bills they have to pay, reminders to go to certain appointments, changes that took place in their group appointments, and so on.
In order to be able to send these notices to all relevant patients at once (e.g. to notify all group members that the meeting location has changed), we need to be able to send messages to multiple patients at once.

sjpadgett · 2017-10-31T17:18:42Z

Will look at more closely. The functionality is similar to my patient/dashboard chat where provider can send a message blast to one or more authorized portal patients. Not so with my secure messaging so, may be able to fit it in there. I have pnotes integrated so would be a matter of adding notification on portal landing page. My secure messaging does allow patient to reply to provider via dashboard where message is screened and forwarded to pnotes if necessary. Could be an interesting addition.

bradymiller · 2017-11-01T09:08:21Z

Also having a tough time getting the offsite patient portal to work (likely since I am using a new development environment and just need to get vpn support up and running); ill keep working on it.

The use case makes sense, especially since we have the Groups module option. Makes sense to get support for this in the new onsite demo ( @sjpadgett 's baby) and also in the offsite demo (I'll try to get ZH involved here).
-brady

shachar058 · 2017-11-06T10:51:17Z

Hi @bradymiller, any news on this?
Will it be possible to bring this in soon?
thanks,
shachar

bradymiller · 2017-11-07T11:14:36Z

hi @shachar058 ,
Sorry for the delay. Been busy with some security issues (these get my high priority), but plan to get back to this over the next 1-2 days.
-brady

bradymiller · 2017-11-08T20:55:43Z

Just tried out this and realized there is now a bug in the way the messaging is working.

There is no longer a way to explicitly send messages to patients (used to be a -Patient- selector), which meant the message was supposed to go to the patient. The other patient selector just means that the message is regarding the patient (does not mean it goes to the patient).

thoughts?

bradymiller · 2017-11-08T21:01:01Z

ok, figuring this out. turns out the patient to selector only shows when turn on the offsite portal:
matrix-israel@eee08a0#diff-413acece90704c1024806070613c874d

@sjpadgett , are you making use of the to patient messaging on this screen. If so, I'll also turn this on when the onsite portal is turned on.

@shachar058 ,
So can clarify the point of your code. Are your messaging meant to be going to the patients. or are they going to another user (but are regarding a group of patients)?

-brady

sjpadgett · 2017-11-08T21:35:03Z

I don't use the flag. I only use the portal-user for forwarded messages from my dashboard to let provider know it's from patient. The patient info is in message content.
So nope, don't use.

shachar058 · 2017-11-09T07:56:24Z

@bradymiller we send it to users regarding patients not to the patients themselves (we don't use the offsite portal), but the user reads the messages in the patient file to the patient.

bradymiller · 2017-11-11T09:09:03Z

I think this is good. In the light of day, just plan to do one more analysis (following reply-to parameter to ensure no conflict between this and ZH's code).

bradymiller · 2017-11-14T01:35:17Z

Light of day analysis looks good and will bring this into the codebase. Thanks for the contribution! -brady

shachar058 added 2 commits October 17, 2017 12:48

enabled multiple patients messaging

3096c8e

some fixes

9caf679

bradymiller added the Status: Needs Review label Oct 18, 2017

bradymiller self-assigned this Oct 19, 2017

bradymiller reviewed Oct 21, 2017

View reviewed changes

bradymiller added the Status: Needs Work label Oct 21, 2017

fix rtrim

ce44b25

bradymiller removed their assignment Oct 23, 2017

bradymiller self-assigned this Nov 11, 2017

bradymiller merged commit 39b7ef6 into openemr:master Nov 14, 2017

shachar058 deleted the messages_to_multiple_patients branch November 22, 2017 10:50

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Messages to multiple patients #1132

Messages to multiple patients #1132

shachar058 commented Oct 17, 2017 •

edited by bradymiller

Loading

bradymiller commented Oct 18, 2017

shachar058 commented Oct 19, 2017

bradymiller commented Oct 19, 2017 •

edited

Loading

bradymiller Oct 21, 2017

shachar058 Oct 22, 2017

bradymiller commented Oct 21, 2017

shachar058 commented Oct 22, 2017

bradymiller commented Oct 22, 2017

shachar058 commented Oct 22, 2017 •

edited

Loading

bradymiller commented Oct 23, 2017

shachar058 commented Oct 24, 2017

bradymiller commented Oct 25, 2017

shachar058 commented Oct 25, 2017

sjpadgett commented Oct 25, 2017

shachar058 commented Oct 29, 2017

bradymiller commented Oct 30, 2017 •

edited

Loading

sjpadgett commented Oct 30, 2017 via email

bradymiller commented Oct 30, 2017

shachar058 commented Oct 31, 2017

sjpadgett commented Oct 31, 2017

bradymiller commented Nov 1, 2017

shachar058 commented Nov 6, 2017

bradymiller commented Nov 7, 2017

bradymiller commented Nov 8, 2017

bradymiller commented Nov 8, 2017

sjpadgett commented Nov 8, 2017

shachar058 commented Nov 9, 2017

bradymiller commented Nov 11, 2017

bradymiller commented Nov 14, 2017

Messages to multiple patients #1132

Messages to multiple patients #1132

Conversation

shachar058 commented Oct 17, 2017 • edited by bradymiller Loading

bradymiller commented Oct 18, 2017

shachar058 commented Oct 19, 2017

bradymiller commented Oct 19, 2017 • edited Loading

bradymiller Oct 21, 2017

Choose a reason for hiding this comment

shachar058 Oct 22, 2017

Choose a reason for hiding this comment

bradymiller commented Oct 21, 2017

shachar058 commented Oct 22, 2017

bradymiller commented Oct 22, 2017

shachar058 commented Oct 22, 2017 • edited Loading

bradymiller commented Oct 23, 2017

shachar058 commented Oct 24, 2017

bradymiller commented Oct 25, 2017

shachar058 commented Oct 25, 2017

sjpadgett commented Oct 25, 2017

shachar058 commented Oct 29, 2017

bradymiller commented Oct 30, 2017 • edited Loading

sjpadgett commented Oct 30, 2017 via email

bradymiller commented Oct 30, 2017

shachar058 commented Oct 31, 2017

sjpadgett commented Oct 31, 2017

bradymiller commented Nov 1, 2017

shachar058 commented Nov 6, 2017

bradymiller commented Nov 7, 2017

bradymiller commented Nov 8, 2017

bradymiller commented Nov 8, 2017

sjpadgett commented Nov 8, 2017

shachar058 commented Nov 9, 2017

bradymiller commented Nov 11, 2017

bradymiller commented Nov 14, 2017

shachar058 commented Oct 17, 2017 •

edited by bradymiller

Loading

bradymiller commented Oct 19, 2017 •

edited

Loading

shachar058 commented Oct 22, 2017 •

edited

Loading

bradymiller commented Oct 30, 2017 •

edited

Loading