ChildMindInstitute / mindlogger-admin-OLD-Vue

Browser-based interface for administering the MindLogger platform
https://admin-prod.mindlogger.org
Other
4 stars 4 forks source link

There is the combined admin and email report after completing the flow with the disallowed "Combine" option #1762

Closed natalia-muzyka closed 2 years ago

natalia-muzyka commented 2 years ago

Expected: Allow an editor at the activity flow level to combine reports from each linked activity into one report. If not combined, then there is one file for each activity in the flow in one email.

Requirements: https://app.zenhub.com/workspaces/mindlogger-webapp-5e11094d0c26311588da9626/issues/childmindinstitute/mindlogger-applet-builder/874

Video: https://www.screencast.com/t/vdQf1bbIxymK

Environment: https://admin-staging.mindlogger.org/ Win 10/ Chrome 103 user-stg@ml.com / 123456 config reprots v11 / my Flow not combined Applet password Qwe123!!!

natalia-muzyka commented 2 years ago

@WorldImpex @jj105 @astef021 is this a bug or acceptable behavior for the Config reports v1 because of the email URL fix?

WorldImpex commented 2 years ago

We should expect with combined on there is 1 URL and with combined off multiple URLs. Could the link in the email send the user to a folder of PDFs to download each one individually?

jj105 commented 2 years ago

I think it's not easy to show folders in s3 bucket. can we send email each time user completes an activity in activity flow in if combine reports is turned off?

WorldImpex commented 2 years ago

Yes that is fine for now @jj105

natalia-muzyka commented 2 years ago

Verified as fixed for Android: https://www.screencast.com/t/Bb7q1eq1Q

Environment: ML v0.21.43 staging Google Pixel 5a // Android 11 user-stg@ml.com 123456 config reports v11 / my flow not combined Applet password Qwe123!!!

natalia-muzyka commented 2 years ago

Verified as fixed for iOS: https://www.screencast.com/t/OqBq4fcl

Environment: ML v0.21.48 staging iPhone 7 // iOS 13.1.1 user-stg@ml.com 123456 config reports v11 / my flow not combined Applet password Qwe123!!!