-
Notifications
You must be signed in to change notification settings - Fork 506
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature Request]: Add a "hint/solution viewed" event to complement the existing "offered"/"unlocked" events. #5069
Comments
@seanlip I would like to work on this. Can you please assign this issue to me. |
@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. |
Hi @seanlip, can you please look at this screen recording and tell me if I am going right? hintandsolutionviews.webm |
@Vishwajith-Shettigar, this LGTM. |
Thank you @adhiamboperes |
Is your feature request related to a problem? Please describe.
We currently have two events for each of hints and solutions, which state:
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
The text was updated successfully, but these errors were encountered: