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

Missing word inside MessageBox's message #29

Closed
jasnow opened this issue Jun 6, 2013 · 12 comments
Closed

Missing word inside MessageBox's message #29

jasnow opened this issue Jun 6, 2013 · 12 comments
Labels
Milestone

Comments

@jasnow
Copy link
Contributor

jasnow commented Jun 6, 2013

If you log in
And click on Gear/About menu option
And click on the first order
And click "Delivered to Hub" radio button
Then you get this message in the "Message Box": "Please pick up your request at this by this date."

Note that it appears to be be missing a word after "this"

This is also true for "Contact PCMO" radiobutton.

If someone tells me what the word is, I will change the code.

@ghost ghost assigned ddeseta Jun 6, 2013
@ddeseta
Copy link

ddeseta commented Jun 6, 2013

This sounds like a UX Issue. The SHORT answer is that the text does not need to be changed right now in either area.

The LONG answer: So here is how the interaction was designed to work. The message in the box was designed to be default text that is to be manually edited by the PCMO before sending it so they can append the message with the actual location name. For phase two we could redesign this feature so that the message automatically calls in the correct location based on a rule set that would determine the nearest location that the medication is in stock. This future interaction design would capture some of the functionality that is needed for RX SOS and would provide a more elegant efficient experience for PCMO's. Let me know if you need more around this.

@jasnow
Copy link
Contributor Author

jasnow commented Jun 6, 2013

What goes between the word this and the word by?

@ddeseta
Copy link

ddeseta commented Jun 7, 2013

I don't have access to the screens we are discussing. I currently have login credentials to "act" like a PCV and see the request form but I need credentials to log in as a PCMO in order to help you with this issue.

@v3n
Copy link
Contributor

v3n commented Jun 7, 2013

@ddeseta, issue #34 is currently blocking you, which I will attempt to get resolved soon.

As this is a UI issue, I additionally moved it to the version 0.2.0 milestone.

@jasnow
Copy link
Contributor Author

jasnow commented Jun 7, 2013

I logged in as rhok@example and saw the problem.

@jasnow
Copy link
Contributor Author

jasnow commented Jun 7, 2013

Took screenshot and saved it as a file "iss29-screenshot.odt" in doc dir.

@ddeseta
Copy link

ddeseta commented Jun 7, 2013

@jasnow and @v3n Thanks :) I'm working on updating the designs with @pstoica for this one.

@ddeseta
Copy link

ddeseta commented Jun 9, 2013

@jasnow, @vsn, @pstoica Please refer to the updated wiki document for all SMS, Web Form and email messages that are triggered when a PCMO fulfills a request. They are located here: https://github.com/atlrug-rhok/medlink/wiki/System-Messages%3A-Positive-Acknowledgements%2C-errors-and-other

@ddeseta ddeseta closed this as completed Jun 9, 2013
@jasnow
Copy link
Contributor Author

jasnow commented Jun 10, 2013

This issue has not been fixed. Please provide the missing word (fix the grammar) between the two boldface words below: "Please pick up your request at this by this date."

@jasnow jasnow reopened this Jun 10, 2013
@jasnow
Copy link
Contributor Author

jasnow commented Jun 10, 2013

After more reading, I found this message which is close: "3) Delivery: Your request is estimated to arrive at your location on this date [enter date here]."

@jasnow
Copy link
Contributor Author

jasnow commented Jun 10, 2013

Changed ""Please pick up your request at this by this date." to "Your request is estimated to arrive at your location on this date [enter date here]." Therefore the missing word is fixed, but TBD for [enter date here] part.

@jasnow
Copy link
Contributor Author

jasnow commented Jun 10, 2013

I will move the "[enter date here] part to another issue and close this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants