oppia / oppia-android

A free, online & offline learning platform to make quality education accessible for all.
https://www.oppia.org
Apache License 2.0
322 stars 523 forks source link

[Feature Request]: Fix event behaviour to properly handle the case when an exploration is resumed #5068

Open seanlip opened 1 year ago

seanlip commented 1 year ago

Is your feature request related to a problem? Please describe.

Some events are not handling the "resuming an exploration" user flow correctly -- either through not recording enough data or by recording data in a confusing way. In particular:

Describe the solution you'd like

Describe alternatives you've considered N/A

adhiamboperes commented 1 year ago

Suggested fix for task 1 and 2:

Change the type of resume_exploration_context in oppia_logger.proto from LearnerDetailsContext to ExplorationContext, and cascade down this change to EventBundleCreator.kt, EventLogSubject.kt and related tests. We would need to move resume_exploration_context to a new field number and reserve 28 for backward compatibility.