ChildMindInstitute / mindlogger-app

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

[PROD] There is a duplicated activity flow in the app in the migrated daisy chain applet #2832

Closed natalia-muzyka closed 1 year ago

natalia-muzyka commented 1 year ago

Preconditions User is logged in on the admin panel https:/admin.mindlogger.org/ bernard_ml@protonmail.com / qwe123!!! Mobile app is installed and launched, user is logged in

Steps to reproduce

  1. Open the app
  2. Proceed to the "daisy chain report" applet
  3. Open admin
  4. Edit the daisy chain applet
  5. Open the Activity flow tab
  6. Compare quantity of flows between builder and app

Actual result There is a duplicated activity flow in the app in the migrated daisy chain applet

Expected result The quantity of flows in the app matches the builder in the migrated daisy chain applet

image.png

Environment: https:/admin.mindlogger.org/ Win 10 / Chrome 103 ML Pilot v0.21.48 prod iPhone 7 // iOS 13.1.1 Google Pixel 5a // Android 11 bernard_ml@protonmail.com / qwe123!!! daisy chain report, daisy chain 0.17.76 Applet password: Qwe123!!!

karser commented 1 year ago

Should be fixed image.png

natalia-muzyka commented 1 year ago

@karser I see the issue is fixed for: the "daisy chain report", daisy chain ML Pilot 0.17.57, daisy chain report (loop) applets,

but is still reproducible in: the "daisy chain 0.17.76", "daisy chain 0.19.25" on the same account

image.png

karser commented 1 year ago

@natalia-muzyka Please check the "daisy chain 0.17.76", "daisy chain 0.19.25" If all is good I will run my script and remove other duplicated flows.

Looks like it shows the flow activity but without the "flow" label. @natalia-muzyka Is this correct behavior?

image.png image.png

natalia-muzyka commented 1 year ago

@karser "daisy chain 0.17.76", "daisy chain 0.19.25" are looking fine. Thank you for noticing the badge issue, we have tickets for it: The "Hide" badge is selected after unselecting it and saving the applet#1048 The "Hide badge" checkbox works in opposite way to its state#2792 The "Hide badge" checkbox is unselected by default#1041

natalia-muzyka commented 1 year ago

@karser I see duplicated flows on the same account (bernard_ml@protonmail.com / qwe123!!!) in the next applets: Cumulatives (Natalia) (full) cumulative retest daisy chain report (loop)

karser commented 1 year ago

@natalia-muzyka I fixed those applets too. The problem was the applets were cached. image.png

natalia-muzyka commented 1 year ago

@karser applets from my last comment are fixed, but I'm still catching duplicated flows on the other accounts. @WorldImpex would you agree to check only the actual applets that are in use by our stakeholders in real programs on prod and fix them only? If yes then, please, share credentials with me.

karser commented 1 year ago

I removed duplicated flows for all DB and refreshed the cache for the related applets. Please check @natalia-muzyka

natalia-muzyka commented 1 year ago

See no duplicated flows now on my accounts. Verified as fixed.

Environment: ML Pilot v0.21.48 Google Pixel 5a // Android 11 Samsung Galaxy Tab S6 SM-T865 // Android 9.0 Apple iPad 9th gen (2021) / iOS 15.5 anita_ml@protonmail.com qwe123!!! bruce_ml@protonmail.com 12345678 bernard_ml@protonmail.com qwe123!!!