Skip to content

Commit

Permalink
Push notifications status analytic (openedx#560)
Browse files Browse the repository at this point in the history
* chore: log push notification status (#102)

* chore: fixes after merge, fix for swift6

---------

Co-authored-by: Saeed Bashir <sbashir@2u.com>
Co-authored-by: Anton Yarmolenko <37253+rnr@users.noreply.github.com>
  • Loading branch information
3 people authored Jan 20, 2025
1 parent 315433f commit e903d39
Show file tree
Hide file tree
Showing 4 changed files with 35 additions and 3 deletions.
3 changes: 3 additions & 0 deletions Core/Core/Analytics/CoreAnalytics.swift
Original file line number Diff line number Diff line change
Expand Up @@ -154,6 +154,7 @@ public enum AnalyticsEvent: String {
case discussionFollowToggle = "Dicussion:Post Follow Toggle"
case discussionLikeToggle = "Discussion:Like Toggle"
case discussionReportToggle = "Discussion:Report Toggle"
case notificationSettingPermissionStatus = "Notification:Setting Permission Status"
}

public enum EventBIValue: String {
Expand Down Expand Up @@ -253,6 +254,7 @@ public enum EventBIValue: String {
case discussionFollowToggle = "edx.bi.app.discussion.follow_toggle"
case discussionLikeToggle = "edx.bi.app.discussion.like_toggle"
case discussionReportToggle = "edx.bi.app.discussion.report_toggle"
case notificationSettingPermissionStatus = "edx.bi.app.notification.setting_permission.status"
}

public struct EventParamKey {
Expand Down Expand Up @@ -308,6 +310,7 @@ public struct EventParamKey {
public static let like = "like"
public static let report = "report"
public static let discussionType = "discussion_type"
public static let status = "status"
}

public struct EventCategory {
Expand Down
13 changes: 12 additions & 1 deletion OpenEdX/Managers/AnalyticsManager/AnalyticsManager.swift
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,8 @@ class AnalyticsManager: AuthorizationAnalytics,
CourseAnalytics,
DiscussionAnalytics,
CoreAnalytics,
WhatsNewAnalytics {
WhatsNewAnalytics,
@unchecked Sendable {

private var services: [AnalyticsService]

Expand Down Expand Up @@ -166,6 +167,16 @@ class AnalyticsManager: AuthorizationAnalytics,
trackScreenEvent(.mainDashboardProgramsClicked, biValue: .mainDashboardProgramsClicked)
}

public func notificationPermissionStatus(status: String) {
trackEvent(
.notificationSettingPermissionStatus,
biValue: .notificationSettingPermissionStatus,
parameters: [
EventParamKey.status: status
]
)
}

// MARK: Discovery

public func discoverySearchBarClicked() {
Expand Down
6 changes: 4 additions & 2 deletions OpenEdX/Managers/AnalyticsManager/MainScreenAnalytics.swift
Original file line number Diff line number Diff line change
Expand Up @@ -8,22 +8,24 @@
import Foundation

//sourcery: AutoMockable
public protocol MainScreenAnalytics {
public protocol MainScreenAnalytics: Sendable {
func mainDiscoveryTabClicked()
func mainLearnTabClicked()
func mainProfileTabClicked()
func mainProgramsTabClicked()
func mainCoursesClicked()
func mainProgramsClicked()
func notificationPermissionStatus(status: String)
}

#if DEBUG
public class MainScreenAnalyticsMock: MainScreenAnalytics {
final public class MainScreenAnalyticsMock: MainScreenAnalytics {
public func mainDiscoveryTabClicked() {}
public func mainLearnTabClicked() {}
public func mainProfileTabClicked() {}
public func mainProgramsTabClicked() {}
public func mainProgramsClicked() {}
public func mainCoursesClicked() {}
public func notificationPermissionStatus(status: String) {}
}
#endif
16 changes: 16 additions & 0 deletions OpenEdX/View/MainScreenViewModel.swift
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ import Course
import Swinject
import Combine
import Authorization
import UserNotifications

public enum MainTab {
case discovery
Expand Down Expand Up @@ -69,6 +70,21 @@ final class MainScreenViewModel: ObservableObject {
}
}
.store(in: &cancellables)
trackSettingPermissionStatus()
}

private func trackSettingPermissionStatus() {
DispatchQueue.global(qos: .userInteractive).async {
UNUserNotificationCenter.current().getNotificationSettings(completionHandler: { [weak self] (settings) in
if settings.authorizationStatus == .notDetermined {
self?.analytics.notificationPermissionStatus(status: "not_determined")
} else if settings.authorizationStatus == .denied {
self?.analytics.notificationPermissionStatus(status: "denied")
} else if settings.authorizationStatus == .authorized {
self?.analytics.notificationPermissionStatus(status: "authorized")
}
})
}
}

public func select(tab: MainTab) {
Expand Down

0 comments on commit e903d39

Please sign in to comment.