Skip to content
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

Android - Add better logging for migration errors #17746

Closed
mkevins opened this issue Jan 12, 2023 · 1 comment · Fixed by #17748
Closed

Android - Add better logging for migration errors #17746

mkevins opened this issue Jan 12, 2023 · 1 comment · Fixed by #17748
Assignees

Comments

@mkevins
Copy link
Contributor

mkevins commented Jan 12, 2023

We are currently only logging a subset of possible errors via Tracks events, but we have a fairly comprehensive set of errors that we can detect. This task is to enable logging of these errors so that more details can emerge from users who wind up in support queues as a result of the migration.

@mkevins mkevins self-assigned this Jan 12, 2023
@mkevins mkevins changed the title Android - Add better logging for errors Android - Add better logging for migration errors Jan 12, 2023
@ovitrif
Copy link
Contributor

ovitrif commented Jan 12, 2023

Adding this reference to the code where migration errors are being handled, where we can start looking into enhancing the logging:

private fun handleErrors(error: LocalMigrationError) {
when (error) {
is ProviderError -> Unit
is Ineligibility -> when (error.reason) {
WPNotLoggedIn -> sharedLoginAnalyticsTracker.trackLoginFailed(WPNotLoggedInError)
LocalDraftContentIsPresent -> migrationAnalyticsTracker.trackContentMigrationFailed(LocalDraftContent)
}
is FeatureDisabled -> Unit
is MigrationAlreadyAttempted -> Unit
is PersistenceError -> Unit
is NoUserFlagsFoundError -> Unit
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants