Skip to content

Commit

Permalink
refactor: Updated event names and bi values
Browse files Browse the repository at this point in the history
Fixes: LEARNER-9941
  • Loading branch information
HamzaIsrar12 committed Apr 24, 2024
1 parent 8d070d0 commit 8022b51
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ enum class CoreAnalyticsKey(val key: String) {
NAME("name"),
CATEGORY("category"),
DISCOVERY("discovery"),
VIDEOS("video"),
VIDEOS("videos"),
PROFILE("profile"),
URL("url"),
ACTION("action"),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ enum class AppReviewAnalyticsKey(val key: String) {
NAME("name"),
CATEGORY("category"),
RATING("rating"),
APP_REVIEW("app_reviews"),
APP_REVIEWS("app_reviews"),
ACTION("action"),
DISMISSED("dismissed"),
NOT_NOW("not_now"),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ open class BaseAppReviewDialogFragment : DialogFragment() {
event = AppReviewAnalyticsEvent.RATING_DIALOG.eventName,
params = buildMap {
put(AppReviewAnalyticsKey.NAME.key, AppReviewAnalyticsEvent.RATING_DIALOG.biValue)
put(AppReviewAnalyticsKey.CATEGORY.key, AppReviewAnalyticsKey.APP_REVIEW.key)
put(AppReviewAnalyticsKey.CATEGORY.key, AppReviewAnalyticsKey.APP_REVIEWS.key)
}
)
}
Expand Down Expand Up @@ -65,7 +65,7 @@ open class BaseAppReviewDialogFragment : DialogFragment() {
AppReviewAnalyticsKey.NAME.key,
AppReviewAnalyticsEvent.RATING_DIALOG_ACTION.biValue
)
put(AppReviewAnalyticsKey.CATEGORY.key, AppReviewAnalyticsKey.APP_REVIEW.key)
put(AppReviewAnalyticsKey.CATEGORY.key, AppReviewAnalyticsKey.APP_REVIEWS.key)
put(AppReviewAnalyticsKey.ACTION.key, action)
rating.nonZero()?.let { put(AppReviewAnalyticsKey.RATING.key, it) }
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ interface DiscoveryAnalytics {
enum class DiscoveryAnalyticsEvent(val eventName: String, val biValue: String) {
COURSE_INFO(
"Discovery:Course Info",
"edx.bi.app.course.info"
"edx.bi.app.discovery.course_info"
),
PROGRAM_INFO(
"Discovery:Program Info",
Expand Down

0 comments on commit 8022b51

Please sign in to comment.