Skip to content

Commit 8331d32

Browse files
committed
Fix: Set reinstallUser flag when sync restore user skips onboarding
The SYNC_RESTORE secondary CTA handler was sending ShowSkipOnboardingOption without setting reinstallUser = true, unlike the analogous INITIAL_REINSTALL_USER handler. A sync restore user necessarily had the app installed before, so they should be treated as a reinstall user. This ensures correct event data when the user later reaches INPUT_SCREEN.
1 parent 47a3750 commit 8331d32

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

app/src/main/java/com/duckduckgo/app/onboarding/ui/page/WelcomePageViewModel.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -266,6 +266,7 @@ class WelcomePageViewModel @Inject constructor(
266266
SYNC_RESTORE -> {
267267
viewModelScope.launch {
268268
logcat { "Sync-AutoRestore: user skipped restore" }
269+
reinstallUser = true
269270
_commands.send(ShowSkipOnboardingOption)
270271
}
271272
}

0 commit comments

Comments
 (0)