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 TEST Dynamic Dashboard #3029

Open ljmoody opened 11 months ago

ljmoody commented 11 months ago

AC:

lijenn commented 11 months ago

@tejaspesquare Please see Laura and my feedback for the dynamic dashboard in Test-Slow:

tejaspesquare commented 11 months ago

@ljmoody @lijenn Point #5 - Decision Meters:- it will enable for input, you need to click on the sleeping line placed inside circle.

Care Coordination: Test instance

image

tejaspesquare commented 11 months ago

@lijenn To answer on Point #9, As per initial estimation feedback loops (casual Flow Diagrams) shall be ready by 20th Dec includes design & development provided no major shuffling on priority tasks. i.e. Amendments on CC,MM & Psychotherapy. New development on loop diagrams on Team Care, Team Flow as per new wireframe.

Flow Meters: As per James, this is suspended until March. So, we have stopped development on this item.

Current In-progress Tasks:

  1. MTL Reports : Preview export Word & PPTx : We are able to export in PPTx & couple of use case yet to develop. POC & follow-up with development on word format is pending. 2.Self Registration Portal: Issue #3026: We are left with just one point no 6. All teams that register in the self-registration must show up users who have the administrative_login privilege. Note: Developer is working on it. 3.Dynamic Dashboard - Loop Diagrams (at moment developer has started on 'Team Care'). 4.Facilitator Dashboard: Popup Window for SME (Subject Matter Expert): Required POC, design & development.
tejaspesquare commented 11 months ago

@lijenn Points #2,3,4,6 are verified & moved to test environment. Point #1 - I had deleted & uploaded existing data file named 'mtl_3.0..jenn_09_22_23_test_slow.xlsx'. This is is working fine from my side.

Points #6 & 7 - development is pending

Note: Jenn removed the team name in the data file. We do not want to name the sites we are working with on our public facing repositories!

lijenn commented 11 months ago

@tejaspesquare I do not see these changes reflected in Test-Slow yet. Were they added to Test-Break-Fix instead?

Regarding point 1, we do not have an issue with uploading the data file into teams. The data file is not loading in the Dynamic Dashboard.

I added the data file named "jenn_test_11_28_23_test_slow.xlsx" into the following worlds:

When we click the green "GO!" button, the page is stuck loading the data into the dynamic dashboard:

These numbers in the dynamic dashboard should be updated after clicking the green "GO!" button:

tejaspesquare commented 11 months ago

@lijenn Changes not moved to test yet. It was up to dev environment. My bad. Per discussion with James on 28th Nov, we are supposed to inform him before any deployment to test environment. We will deploy in test can you check with him & confirm.

Reg. loading data file issue on Dynamic Dashboard, we have also experienced the same for facilitator & administrator world login. It on our to do list & with priority. Actually, developer is on planned leave (28th Nov to 30th Nov) hence no progress on it.

tejaspesquare commented 11 months ago

@lijenn Changes moved to TEST-Slow. Kindly test & provide your comments. Reg. loading data file issue on Dynamic Dashboard - We will have to connect with David to rectify issue on API results for 2 different worlds.

tejaspesquare commented 11 months ago

@ljmoody @lijenn Point no 1 is working now. On click 'Go' the data file is loading in the Dynamic Dashboard & numbers are updated too. Changes moved to TEST-Slow instance. Kindly check & let us know if issue still persist.

lijenn commented 11 months ago

Thanks @tejaspesquare! Regarding point 5 Decision Meters, is there any way to either increase the sleeping line placed inside the circle and/or have the sleeping line be automatically pre-clicked so when the user opens the decision meter, they can more easily enter the text?

Our users busy and stressed leaders and front-line staff so the current design may frustrate them.

Pre-clicked:

image
tejaspesquare commented 11 months ago

Thanks @tejaspesquare! Regarding point 5 Decision Meters, is there any way to either increase the sleeping line placed inside the circle and/or have the sleeping line be automatically pre-clicked so when the user opens the decision meter, they can more easily enter the text?

Our users busy and stressed leaders and front-line staff so the current design may frustrate them.

Pre-clicked: image

@lijenn - Ok. We will check for possibility what we can do for better experience.

lijenn commented 11 months ago

Hi @tejaspesquare! Any updates on 5, 7, and 8?

tejaspesquare commented 10 months ago

Hi @tejaspesquare! Any updates on 5, 7, and 8?

@lijenn - Point 5 is fixed & available on TEST-Slow. Please check & let us know if any changes required. Point 7 & 8 shall be ready by Wednesday EOD (3rd Jan'24).

ljmoody commented 10 months ago

Hi, @tejaspesquare. @lijenn and I have some more follow-ups on this card, as follows:

New tasks:

tejaspesquare commented 10 months ago

@ljmoody - Point No 5 Decision meters input fields: This is fixed & moved to Test-slow. Please note input boxes will present blank by default unless user filled in some value & save it.

tejaspesquare commented 10 months ago

@ljmoody - Please find status as per following & done part moved to TEST-Slow. Done -

  1. Line delineations - When the module sections are contracted
  2. All stock numbers should present completely within the blue stock background - this one is beyond the edge:
  3. We noticed the sideways caret is positioned above the maximum line.
  4. Ensure font size for all numbers in all stocks are the same:

    Pending -

  5. Visual alignment. - Align all stock boxes (left and right sides should align, as well as top/bottom) for all modules:
ljmoody commented 10 months ago

Hi, @tejaspesquare. Jenn and I did more user testing and comparing against the wireframes and have feedback to incorporate. FYI @lijenn

tejaspesquare commented 10 months ago

@ljmoody - Please see my response

  1. Point 13 - Moved to TEST-Slow
  2. Point 23 Change behavior between administrator and facilitator logins for MTL Reports screen as shown:- At this moment this development is not possible. However, we will give a try to find solution.
  3. Points 14 to 20, & 22 - Currently, team is occupied in Flow meters (CC, MM & Psychotherapy w.r.t latest wireframe) development on priority. Hence, we will attend those after 16th or 17th Jan.
  4. Point 21 - causal loop diagrams: show entire feedback loop without having to scroll :- We will check whether it is possible or not.
tejaspesquare commented 9 months ago

@ljmoody - Point 21 - causal loop diagrams: show entire feedback loop without having to scroll :- It is not possible to show up loop diagrams without scroll on popup unless we open in a different tab/window. Point 23 - Change behavior between administrator and facilitator logins for MTL Reports screen as shown:- This is not possible to implement. We have to rewrite entire code (end-to-end) but there is no guarantee it will be a success. Points 14 to 20 & 22 - We will start working on these observations from today (16th Jan).

FYI @jamesmrollins & @lijenn

tejaspesquare commented 9 months ago

Hello @ljmoody Please find status as given below: Points done & moved to Test-Slow - 13,14,16,17,18,19,22 Pending -15

FYI @jamesmrollins & @lijenn

ljmoody commented 9 months ago

Hi @tejaspesquare - is there an update on the remaining tasks associated with this issue card?

tejaspesquare commented 9 months ago

@ljmoody - Point 15, 23 are fixed & available to test on TEST-Slow instance.
Point 20 (Casual Loop without Scroll) - This is in-progress.

tejaspesquare commented 9 months ago

@ljmoody & @lijenn: Point 20 & 21 (Casual Loop without Scroll) - This is fixed for remaining models & it is available to test in TEST-Slow. You can continue testing on this environment too.

ljmoody commented 9 months ago

@tejaspesquare - please incorporate these changes to the DDD:

FYI @lijenn

ljmoody commented 8 months ago

Hello @tejaspesquare - please see the wireframe_DDD tutor.pdf for the Dynamic Decision Dashboard tutor prompt wireframe.

ljmoody commented 8 months ago

Based on feedback received today, please incorporate the following changes to the DDD @tejaspesquare: The desired behavior is to show, by default, the array menus. Users will need to click or change radio button selections to close. Please check boxes as development is complete.

image

tejaspesquare commented 8 months ago

@ljmoody - The requested array menus changes quite difficult to develop. The entire UI will get disturbed. Partial stock boxes will be appeared in background of this menu.

image

Alternate solution if it works: Put radio button selection outside like below.

image

Note: This requires significant efforts in terms of design.

ljmoody commented 8 months ago

@tejaspesquare, thanks for your feedback about the array changes.

Of the two solutions you identified above (overlapping the stocks or placing separately), would one be easier than the other or take less time to develop? If you could give an estimate of time/work involved, we can share with Lindsey and that would help decision-making.

Also, how do you propose Team Care (see screenshot) would be handled if the option to placing the array items separately was used (because there's more options in this array than any other)? image

tejaspesquare commented 8 months ago

@jamesmrollins , @ljmoody , @lijenn - Please find mock for array menus & share your view.

image

Note: Please Ignore, alignment, font size & browser Zoom level. We will start development after your feedback.

tejaspesquare commented 8 months ago

@ljmoody - Point #24. In the test environment, using VA laptop, Edge browser, zoomed at 90%, one of the Team Flow stocks bumps to a second line - This is fixed & moved to TEST-Slow. Please verify.

ljmoody commented 8 months ago

Hi @tejaspesquare - I will meet with Jenn today to review the mockup of the arrays and get back to you about those soon. We may also need to meet with Lindsey to discuss what she would like to see.

Point 24 is verified, looks much better - thank you! image

ljmoody commented 8 months ago

Hello @lzim - could you please review this mockup of the array menus and give your feedback? Please disregard alignment, font size & browser zoom level; this is a mockup provided by dev. image

ljmoody commented 8 months ago

@tejaspesquare - please see wireframe_Episodes of Care.pdf for the MM and Psych Episodes of Care tutor prompt wireframe and edits to these windows.

Please check off as they're developed and ready to test:

fyi: @lijenn

lzim commented 8 months ago

Hi, thanks everyone for all the hard work on this! 😅 In terms of delays and costs, it's too bad we can't simply keep the array menu open in dashboard default view until the user closes it, so that the user can see which array selection is being displayed. The fact that it covers the stocks until it is closed is a feature not a bug.

But, if you've found a re-design is necessary to clarify which array is displayed in the stocks of the dashboard, and we are ignoring for the lack of alignment in the mockup. Then the biggest remaining issue is the abbreviations for the Team Care Array:

For users to interpret what they mean the abbreviations need to be the following:

The MM and Team Flow arrays for other modules are fine - and the HOVER reveal is elegant an elegant solution. 👍🏽

Thanks!

ljmoody commented 8 months ago

Hi @tejaspesquare and team! Before your weekend begins, we'd also like an update on this issue (progress, level of effort, estimate to complete, etc.). Thank you~

ljmoody commented 8 months ago

3/1/24 update from Tejas:

  • Changes on Array menus - Development not started yet.
ljmoody commented 8 months ago

@tejaspesquare - FYI, there will be one more tutor prompt wireframe to complete the DDD tutor prompt wireframes-for the decision meters (one for each module). I estimate getting that to you sometime next week.

fyi: @lijenn

tejaspesquare commented 8 months ago

@ljmoody , @lijenn - Array menu changes done on Dynamic Dashboard. Kindly check on SIM-Dev environment & provide your feedback. We will move code to TEST-Slow once you are ok with it.

image

ljmoody commented 8 months ago

Hi @tejaspesquare - that's good progress! Please incorporate this feedback and let us know when it's available in TEST-Slow. image fyi: @lijenn

ljmoody commented 8 months ago

Hi @lijenn - wanted it to be on both our radars to review decision meter draft tutor prompts so I can finalize and give to Tejas. Linked here for your convenience.

tejaspesquare commented 8 months ago

@ljmoody - Regarding Array items for MM & Team Flow: Putting name description won't be difficult but it will enforce 2 more rows (1 for each) need to be created as it can't be fitted in a single row. Also, as per team there will be designing issue for entire frame (Dynamic Decision Dashboard) for this change.

ljmoody commented 8 months ago

@tejaspesquare - I miscommunicated what we would like to change. Please see the updated graphic in my message to you yesterday, which may help. The acronym lists and radio buttons work well as they are. We would like to change hover text that shows so the hover text defines the acronym definition, but does not include the acronym in parentheses.

tejaspesquare commented 8 months ago

@ljmoody - Can you please check now. We have removed acronym while doing hovering, it only contains definition of item.

ljmoody commented 8 months ago

Hi @tejaspesquare - yes, this is the expected behavior. Thank you. Here's a screen capture video that shows what I see/experience.

FYI: @lijenn, @jamesmrollins

tejaspesquare commented 7 months ago

@jamesmrollins , @ljmoody , & @lijenn - FYI Tutor development (wireframe_DDD tutor.pdf) is done & it is available for testing on TEST-Slow. Please be informed we also doing testing in parallel.

ljmoody commented 7 months ago

@tejaspesquare, I looked at the DDD Tutor prompt development. It looks good and because of changing the arrays to radio buttons, a few tag placements and messages need to be changed now:

image fyi @lijenn

tejaspesquare commented 7 months ago

@ljmoody , @lijenn :- wireframeEpisodes of Care tutor#3029, This is available to test on TEST-Slow. Please note, we are doing parallel testing from our side.

ljmoody commented 7 months ago

Thank you, @tejaspesquare, I hope to validate the Episodes of Care tutor prompts today. FYI, there is still one more DDD tutor prompt wireframe to complete - for decision meters (one for each module). We have focused our efforts on testing lately; I hope to have that document to you this week.

ljmoody commented 7 months ago

@tejaspesquare, see comments below for suggested revisions to Episodes of Care tutor prompts & "i" icon behavior:

image

image

ljmoody commented 7 months ago

@tejaspesquare,

tejaspesquare commented 7 months ago

@ljmoody , @lijenn - Please find updates.

  1. Now, click on 'i' icon brings up single window. Kindly verify on TEST-Slow. image

  2. It is not possible to apply hyperlink for "CPT Code Cheatsheet" image

  3. Rename done - "Facilitator Dashboard" to "Administrator Dashboard" image

Thank You!