search

EmotionCognitionLab / factorial-design-breathing-study

1 stars 0 forks source link

Earnings may skip session payments #20

Closed noah10 closed 1 week ago

noah10 commented 3 weeks ago

At 12:47PM on 6/25, Paul reported that his earnings showed a bonus payment for the day but no session payments:

image At the end of the day on 6/24 is when we write the first earnings records for him:

2024-06-24T21:46:48.012Z INFO About to save 3 earnings records and 6 sessions for user ********.

Those three earnings records were a breath1, a p_breath2 and a visit1.

But then when he uploads his first session on 6/25 we save the session but don't give him any earnings for it:

2024-06-25T15:36:59.560Z INFO About to save 0 earnings records and 1 session for user *******.

This feels like it might be some sort of date math bug around here, or possibly something wrong with the value we fetched for latestTimeEarnings.

juliana-20 commented 3 weeks ago

This also occurred for me. There are no payments under the breathing session column.

choipaul92 commented 2 weeks ago

Update:

At 8:32AM on 6/26 (Day 3 of pilot testing), my bonus payment for the day is displayed but no session payments:

Note: At this time of update, all 18-minute sessions have been completed for Day 1 and Day 2. No sessions have been started as of this time.

IMG_1404

juliana-20 commented 2 weeks ago

Update - I have completed my first session on Day 3. It looks like I received payment under the breathing session column and bonus column, which is correct. Screenshot 2024-06-26 091621

noah10 commented 2 weeks ago

This should now be fixed. Testing it will be a two-day process: You will need to do two complete sessions on day X and make sure that you get credit in the "Breathing Sessions" column for both of them. You will then need to do another session on day X+1 and make sure that you get credit in the "Breathing Sessions" column for that. (You may or may not get bonus credits for any of these, but that's not related to this bug.)

choipaul92 commented 1 week ago

Ver: 1.0.5.

Tested across 3 days with a new condition, ensuring no earnings were skipped on the dashboard or session messages. The earnings dashboard was aligned with earning messages, and rewards calculations were double-checked using Excel.

Note: I cannot close this issue because I am not the author