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
15 stars 23 forks source link

2025_02: TEST Reports Generator #3028

Open ljmoody opened 1 year ago

ljmoody commented 1 year ago

AC:

To Test:

Decisions:

ljmoody commented 10 months ago

Hello @tejaspesquare. Jenn and I tested the reports generator and have the following feedback after today's session:

FYI @lijenn & @jamesmrollins.

tejaspesquare commented 10 months ago

@ljmoody - Please see my response

Point 2 - Edits (3 total) to MTL Reports pop-up window:- i. Removing drop-down is not possible (Select Team)
Point 3 - Two changes to font size offerings and default:- By default it will Small, user wants then need select Normal explicitly. Point 5 - Scrolling modifications within MTL Reports pop-up window: - yes, popup window size is very much possible, but it won't completely remove scrolling. Point 8 - Additional checkboxes needed under "Select Causal Loop Diagrams" and "Select Flow Meters":- Sections Loop Diagrams, Flow Meters & other details are displayed based on drop-down selection of Group Name & Module from top. It is not possible to present all in just a single combination. Point 9 - MTL Reports preview/save issue + italicize "MTL": - Press 'Save' will save details only whereas on click 'Preview' will save & show saved details. We have set delay time of 10secs so that all selected data save properly. Point 10 - Three (3) corrections on generated report output: - Services/Settings -> It will represent information related top service by default & user have provision to change it. Do we really need to restrict user to fetch details of other services?
Point 13 - Three (3) suggested improvements to the MTL Reports pop-up window behavior and headings: It will erase all filled details of previous combination after toggling to different module. Point 14 - Report does not export to Word: This is working at our end. Can you please share downloaded file to verify at our end. Point 15 - Scrolling behavior in Report Preview affects Generated Report output: This is not possible to change its behavior because limitation of editor. Point 16 - James informed us that users should be able to edit/revise the text in the Word or PPT export. Any updates on this feature? : - We tried to make it happen but unfortunately not found any solution.

FYI @lijenn & @jamesmrollins

tejaspesquare commented 10 months ago

@ljmoody - Please find status:

Points done - 1,2,3,4,5,6,7,8,9,10,11,12,13,15 Point 14 - Report does not export to Word: If issue persist then please share downloaded document with us.

FYI @jamesmrollins & @lijenn

lijenn commented 10 months ago

@tejaspesquare Attached is the Word file. The feature generates this file, but it is corrupted and an error message pops up when trying to open it. MTL_Reports_sp_09_15_jenn_friday_test_team_2024_1_17.docx

tejaspesquare commented 10 months ago

@lijenn - I'm able to download & view attached word file. image image image

lijenn commented 10 months ago

@tejaspesquare What program/app are you using to open the file? Our users use Microsoft Word. I get this error when opening the file in Microsoft Word:

image
tejaspesquare commented 10 months ago

@lijenn - We have WPS office installed on our machine. There is an issue while opening with Microsoft office setup. We will fix this problem & will let you know.

ljmoody commented 10 months ago

@tejaspesquare -

fyi @jamesmrollins @lijenn

ljmoody commented 9 months ago

Hello @tejaspesquare

from Praxis Teams 2/5/24: MTL Reports: Viewing Downloaded Report (Word file): This is partially resolved. Can you please verify in development environment. Viewing doc in Print Layout having some issue but you switch to Web Layout it should work as expected. Also, covered red asterisks Recommendation & Experiments fields. Note: Team Care having some header issue, that we will look into it**

I tested in DEV environment today and this is what I experienced:

  1. MTL Report exported to Word, print view: image

  2. MTL Report exported to Word, web view: image

  3. MTL Report exported to PowerPoint view: image

  4. Your revision to the Selected Reports area looks good!: image

fyi @jamesmrollins @lijenn

tejaspesquare commented 9 months ago

@ljmoody - You see PROTECTED VIEW due to settings applied in File properties. Click on 'Enable Editing' will allow to view file content. To fix it permanently please apply following solution in Word: Go to File > Options. Select Trust Center > Trust Center Settings > Protected View. Clear the box for Enable Protected View for files originating from the internet.

FYI - @lijenn

ljmoody commented 9 months ago

@tejaspesquare, Thank you for pointing that out and giving me instructions! I did it and can now see the Word report. I'll check the functionality of it with Jenn and give any feedback we have after that.

ljmoody commented 9 months ago

Hello, @tejaspesquare - please see the wireframe_MTL Reports tutor.pdf for the MTL Reports tutor prompt wireframe.

ljmoody commented 9 months ago

Hello @tejaspesquare. Jenn and I tested again today and have this feedback to offer:

lijenn commented 9 months ago

Hi @tejaspesquare Here is the .doc and .pptx files for the 09_15_jenn_friday_test_team in TEST.

The .doc file does not show any output (I will send in Teams, GitHub does not allow me to upload). How hard would it be to export using .docx instead of .doc for MTL Reports?

The image scale in the .pptx is too large/stretched. MTL_Reports_cc_09_15_jenn_friday_test_team_2024_2_15.pptx

tejaspesquare commented 9 months ago

@lijenn - At this juncture it looks like time consuming to fix issue for .docx version. However, we will give a try sometime later once time permit.

One more update regarding Recommendations section: We are able to represent this block in editable mode however couldn't find a way to bring logo & design formats applied for rest of sections in a report.

Please have a look below, image

For your review, tomorrow we will publish it on Dev environment (https://forio.com/app/va/va-psd-dev/login.html).

FYI @jamesmrollins, @ljmoody

tejaspesquare commented 9 months ago

@jamesmrollins , @ljmoody , & @lijenn - Points #19 To #24 are resolved & moved to TEST-Slow. Please verify & let us know your feedback. Point #23- Formatting of exported document works with all features except underline, Highlight & text color.

Report Recommendations: We currently observe a discrepancy between the icon and box appearance in relation to the UI of other sections in the exported document. It is my understanding that aligning the UI is not a current priority for us. However, we are committed to addressing these UI issues at our earliest convenience.

image

ljmoody commented 9 months ago

@tejaspesquare - Hi there! I used TEST instance to review the following points and have this feedback to offer: (fyi @lijenn, @jamesmrollins)

This is what the generated PPT looks like now: image

ljmoody commented 9 months ago

@tejaspesquare - I updated the link to the wireframe_MTL Reports tutor.pdf. You should have access now.

tejaspesquare commented 9 months ago

@ljmoody - Please see my response

Point #20: We will change text as mentioned. Point #21: Recommendation box - i. We will remove gray background color from text field ii. Heading font face will appear for Recommendations as Word document settings in user's system iii. Exporting to word: Can you please change view of the document from Main menu (View). In our system we are able to see it properly.

image

Point #22: Hyperlink - This will only work if you export a document (word). For PPTx it is not possible as it always rendered as image. Point #23: Formatted document to report: underline, Highlight & text color these three not going to work. Point #28: At present logo is rendered on top section only, can you please clarify.

ljmoody commented 9 months ago

@tejaspesquare - regarding points #21, 23, 28: We spoke with James today and he would like to discontinue the Word and PowerPoint reports. Instead, is it possible to:

fyi: @lijenn

ljmoody commented 8 months ago

Hi @tejaspesquare and team! Before your weekend begins, please give us 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:

  • Export PDF report: Due to sync up issue between Github & Epicenter changes are not reflected. Hence, can't perform internal testing & unable to move code for your testing too.
tejaspesquare commented 8 months ago

@ljmoody , @lijenn - Export to PDF is available to test on SIM-Dev environment. Kindly verify & provide your feedback.

image

image

image

ljmoody commented 8 months ago

@tejaspesquare - we'll meet with Lindsey tomorrow and provide feedback based on decisions made with her input.

To help us prep for that meeting:

tejaspesquare commented 8 months ago

@ljmoody - PDF Report: It is not recommended to have all images one after other without page breaking. If we do so this will make page getting shrink while doing print & even it won't be available in proper readable format post print. However, if this is called for then required POC (proof of concept) & after it can derive actual efforts.

ljmoody commented 8 months ago

Thanks, @tejaspesquare, for the information about the PDF report and pages. We will discuss and let you know if a POC or development is needed.

tejaspesquare commented 8 months ago

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

ljmoody commented 8 months ago

Hi @tejaspesquare, I started testing this today and then Test-slow began behaving oddly. I'm unable to see Tutor prompts past the first couple now. Because the MTL Reports function has been deprioritized and moved forward to late April and beyond, and may likely also change somewhat, I am going to hold off on reviewing these Tutor prompts.