Skip to content

Commit

Permalink
Merge pull request #293 from Automattic/update/track-app-exit-paths-f…
Browse files Browse the repository at this point in the history
…rom-account-details

Track actions that exit app from AccountDetailsFragment
  • Loading branch information
ashiagr authored Sep 5, 2022
2 parents d2835b4 + 15026c0 commit d9bc265
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 0 deletions.
1 change: 1 addition & 0 deletions modules/features/profile/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ dependencies {
implementation project(':modules:services:localization')
implementation project(':modules:services:preferences')
implementation project(':modules:services:utils')
implementation project(':modules:services:analytics')
implementation project(':modules:services:images')
implementation project(':modules:services:ui')
implementation project(':modules:services:compose')
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,8 @@ import androidx.fragment.app.viewModels
import au.com.shiftyjelly.pocketcasts.account.AccountActivity
import au.com.shiftyjelly.pocketcasts.account.ChangeEmailFragment
import au.com.shiftyjelly.pocketcasts.account.ChangePwdFragment
import au.com.shiftyjelly.pocketcasts.analytics.AnalyticsEvent
import au.com.shiftyjelly.pocketcasts.analytics.AnalyticsTrackerWrapper
import au.com.shiftyjelly.pocketcasts.compose.AppTheme
import au.com.shiftyjelly.pocketcasts.compose.components.HorizontalDivider
import au.com.shiftyjelly.pocketcasts.models.to.SignInState
Expand Down Expand Up @@ -56,6 +58,7 @@ class AccountDetailsFragment : BaseFragment() {
@Inject lateinit var userManager: UserManager
@Inject lateinit var playbackManager: PlaybackManager
@Inject lateinit var podcastManager: PodcastManager
@Inject lateinit var analyticsTracker: AnalyticsTrackerWrapper

private val viewModel: AccountDetailsViewModel by viewModels()
private var binding: FragmentAccountDetailsBinding? = null
Expand Down Expand Up @@ -182,10 +185,12 @@ class AccountDetailsFragment : BaseFragment() {
}

binding.btnPrivacyPolicy?.setOnClickListener {
analyticsTracker.track(AnalyticsEvent.ACCOUNT_DETAILS_SHOW_PRIVACY_POLICY)
context?.startActivity(Intent(Intent.ACTION_VIEW, Uri.parse(Settings.INFO_PRIVACY_URL)))
}

binding.btnTermsOfUse?.setOnClickListener {
analyticsTracker.track(AnalyticsEvent.ACCOUNT_DETAILS_SHOW_TOS)
context?.startActivity(Intent(Intent.ACTION_VIEW, Uri.parse(Settings.INFO_TOS_URL)))
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -74,4 +74,8 @@ enum class AnalyticsEvent(val key: String) {
/* Account Updated */
ACCOUNT_UPDATED_SHOWN("account_updated_shown"),
ACCOUNT_UPDATED_DISMISSED("account_updated_dismissed"),

/* Account Details */
ACCOUNT_DETAILS_SHOW_PRIVACY_POLICY("account_details_show_privacy_policy"),
ACCOUNT_DETAILS_SHOW_TOS("account_details_show_tos"),
}

0 comments on commit d9bc265

Please sign in to comment.