lzim / teampsd

Team PSD is using GitHub, R and RMarkdown as part of our free and open science workflow.
GNU General Public License v3.0
9 stars 23 forks source link

2024_10: DEV & TEST Flow tutor prompts #3202

Open ljmoody opened 4 months ago

ljmoody commented 4 months ago

Resources:

Dev AC:

Tutor prompts are developed for all 5 modules in order of priority:

Testing AC:

matthewtomo commented 2 months ago

week 3 2024_08 Praxis Meeting @matthewtomo @jamesmrollins @ljmoody

matthewtomo commented 2 months ago

Discussed Dev Standup wk_1 2024_9 @matthewtomo @tejaspesquare

ljmoody commented 2 months ago

Discussed at wk1 2024_09 sim_ui_workflow @lzim @matthewtomo @jamesmrollins @ljmoody

Dev - to note:

tejaspesquare commented 2 months ago

Hello @ljmoody ,

Dev - to note:

  • Revise key title on all module flow diagrams per Lindsey's request:

Title has been renamed for all models as requested & can be verified on DEV.

image

ljmoody commented 2 months ago

✅ verified revised key title in DEV instance 🚀!

Title has been renamed for all models as requested & can be verified on DEV.

image

matthewtomo commented 1 month ago

Discussed Dev Standup wk_1 2024_9 @matthewtomo @tejaspesquare

tejaspesquare commented 1 month ago

Hello @ljmoody ,

  • @tejaspesquare - All flows (pin 8) and Team Flow prompts (pins 3 and 8) show several sentences with space in-between. Please keep the space between lines as shown. If that's not possible, please let me know I can provide an alternative.

Please be informed that it is not possible for us to maintain the spacing between lines for pins 3 and 8 as specified in the wireframe, as this matter is beyond our control.

image

image

Kindly suggest an alternative.

ljmoody commented 1 month ago

@tejaspesquare - Okay, an alternative to the spacing is to begin each sentence on a new line; that would at least break up what currently looks like a paragraph. Is that possible?

fyi @matthewtomo

Please be informed that it is not possible for us to maintain the spacing between lines for pins 3 and 8 as specified in the wireframe, as this matter is beyond our control.

image

image

Kindly suggest an alternative.

matthewtomo commented 1 month ago

Discussed Dev Standup wk_3 2024_9 @matthewtomo @tejaspesquare

ljmoody commented 1 month ago

@matthewtomo @tejaspesquare Another idea (if sentences cannot begin on a new line) would be something like below, with two spaces, a tilde (or dash or character), two spaces, then the following sentence.

This depicts care inflows and outflows. ~ Red indicates accumulations of patients waiting to start, patients early in care, and patients waiting to step up or down in care. ~ Blue indicates steady flow in and out of that state of care.

tejaspesquare commented 1 month ago

Hello @ljmoody , @matomasz ,

@matthewtomo @tejaspesquare Another idea (if sentences cannot begin on a new line) would be something like below, with two spaces, a tilde (or dash or character), two spaces, then the following sentence.

This issue has been addressed and can be confirmed on the Development instance.

Ex. Team Care.

image

This change should reflect on other modules tutor prompt as well.

ljmoody commented 1 month ago

✅ verified solution in DEV instance - and it's just what Lindsey asked for - yay! 🎉

fyi @matthewtomo @tejaspesquare

ljmoody commented 1 month ago

9/26/24 testing results

fyi @matthewtomo

All 5 modules

Team Care Flow

Team Flow Flow

matthewtomo commented 1 month ago

Discussed Dev Standup wk_1 2024_10 @matthewtomo @tejaspesquare

  • [x] Diagram does not match Tutor wording: image
ljmoody commented 1 month ago

@matthewtomo, agreed and I'll make your suggested Tutor revisions (and others I can make) via Admin login>Tutor in TEST instance and check them off as completed. My Friday brain wasn't thinking I could make changes to those...

  • I think it would be better to adjust the tutor text, rather than updating the order of the flows (which would likely take more work than updating the text).
  • Proposed text change: "The width of the flow lines under this pin are proportional to the flow quantity. To aid in comparing inflows & outflows, they appear in the same fixed order."
ljmoody commented 1 month ago

https://github.com/lzim/teampsd/issues/3261#issuecomment-2393600938:

Discussed Dev Standup wk_1 2024_10 @matthewtomo @tejaspesquare

  • Tutor prompt 4 for team care and team flow positioning has been resolved

fyi: @ljmoody

lzim commented 1 month ago

Great recommendation to edit the text rather than more DEV @matthewtomo Follows the point I was making about the chat (I think)... 👍🏽

matthewtomo commented 2 weeks ago

Discussed Dev Standup wk3 2024_10 @matthewtomo @tejaspesquare

matthewtomo commented 2 weeks ago

Discussed Friday Dev Standup wk3 2024_10 @matthewtomo @tejaspesquare

matthewtomo commented 2 weeks ago

Discussed Dev Standup wk4 2024_10 @matthewtomo @tejaspesquare

lzim commented 1 week ago

@matthewtomo Who is "Designer" and who is "Developer" in post above? Thx!

matthewtomo commented 1 week ago

@matthewtomo Who is "Designer" and who is "Developer" in post above? Thx!

Hi @lzim,

Designer and Developer are members of our development team in India.

tejaspesquare commented 1 week ago

Hello @matthewtomo ,

  • For team flow, tutor prompts 8 and 9 do not aim at the correct elements on the page

This has been fixed now & available for review on Dev instance. image image