ChildMindInstitute / mindlogger-app

MindLogger (React Native) data collection app
Other
15 stars 6 forks source link

The "Show report at the end of the activity" checkbox is allowed by default #2771

Closed natalia-muzyka closed 2 years ago

natalia-muzyka commented 2 years ago

Steps to reproduce

  1. Open a site https://admin-staging.mindlogger.org/
  2. Create a new applet with a builder
  3. Add a new activity
  4. Open the Reports tab on the activity level
  5. Pay attention to the screen

Actual result The "Show report at the end of the activity" checkbox is allowed by default

Expected result The "Show report at the end of the activity" and "Make Report available for Export" checkboxes are in greyed out state until:

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

Environment: https://admin-staging.mindlogger.org/ v0.17.9 Win 10 // Chrome 103

natalia-muzyka commented 2 years ago

@binaryExDev Right after selecting "show report" and moving to the other tab or saving the applet, the checkbox will look unselected after opening the reports the next time. On the mobile app side summary will be shown. Video: https://www.screencast.com/t/DPGWNr6Z91

Environment: https://admin-staging.mindlogger.org/ v0.17.11 Win 10 // Chrome 103 user-stg@ml.com / 123456 config reports 8 / test A

MaksimYurys commented 2 years ago

Checkbox status bug confirmed as fixed.

Video: https://images.zenhubusercontent.com/103405139/2e1b0684-996f-4c42-b505-398b8e8d4926/b7a3c178_4a03_4c86_9ae0_b5a8a58ef0a0.mp4

Environment: https://admin-staging.mindlogger.org/ v0.17.14 Win 10 // Chrome 103 JhoneeMind@gmail.com // Qqww!123 ConfReports 3 // Test 3 aplet password - Qqww!123

MaksimYurys commented 2 years ago

@devbtech the main bug is reproduced.

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

https://admin-staging.mindlogger.org/ v0.17.14 Win 10 // Chrome 103 JhoneeMind@gmail.com // Qqww!123 ConfReports 3 // Test 3 aplet password - Qqww!123

natalia-muzyka commented 2 years ago

@devbtech the issue is fixed for the new applet, but reproducible for legacy applets video: https://www.screencast.com/t/qJdqUHSCJMrk

Environment: https://admin-staging.mindlogger.org/ Win 10 / Chrome 103 loui_ml@protonmail.com / 12345678 daisy chain loop, daisy chain test, CA Partners Applet 123 Applet password Qwe123!!!

natalia-muzyka commented 2 years ago

Verified as fixed: https://www.screencast.com/t/GfUimUMYVe

Environment: https://admin-staging.mindlogger.org/ Win 10 / Chrome 103 loui_ml@protonmail.com / 12345678 daisy chain loop, daisy chain test, CA Partners Applet 123 Applet password Qwe123!!!