oppia-android icon indicating copy to clipboard operation
oppia-android copied to clipboard

[Feature Request]: Add a "hint/solution viewed" event to complement the existing "offered"/"unlocked" events.

Open seanlip opened this issue 1 year ago • 5 comments

Is your feature request related to a problem? Please describe. We currently have two events for each of hints and solutions, which state:

  • When the hint/solution was first offered
  • When the hint/solution was first unlocked

Describe the solution you'd like The aim of this issue is to add a third "hint/solution viewed" event, to handle the case where the learner refers to a hint after initially unlocking it. Unlike the other two events, this event can trigger more than once for each hint/solution (the other two only trigger at most once in an "exploration session").

The first view of a hint/solution (immediately after it has been unlocked) should count as a view here.

If possible, it would be nice to also send the view_count with the "hint/solution viewed" event (e.g. is this the first view, the second view, the fifth view, etc.) but this is optional.

Describe alternatives you've considered N/A

seanlip avatar Jun 19 '23 17:06 seanlip

@seanlip I would like to work on this. Can you please assign this issue to me.

Chandra-Mauli-Sharma avatar Jun 19 '23 19:06 Chandra-Mauli-Sharma

@Chandra-Mauli-Sharma This issue does require a bit of knowledge about this code base, but if you can provide a brief description of how you would go about it, we can assign it to you.

My recommendation though, would be to pick an issue from here that is not currently assigned to someone.

adhiamboperes avatar Jun 19 '23 19:06 adhiamboperes

Hi @seanlip, can you please look at this screen recording and tell me if I am going right?

hintandsolutionviews.webm

Vishwajith-Shettigar avatar Jan 19 '24 15:01 Vishwajith-Shettigar

@Vishwajith-Shettigar, this LGTM.

adhiamboperes avatar Jan 19 '24 16:01 adhiamboperes

Thank you @adhiamboperes

Vishwajith-Shettigar avatar Jan 19 '24 16:01 Vishwajith-Shettigar