Do not suggest SMS during onboarding.

This commit is contained in:
Greyson Parrelli 2022-03-16 14:49:37 -04:00 committed by Cody Henthorne
parent da1ac5358f
commit 4abb169568
2 changed files with 4 additions and 4 deletions

View file

@ -47,7 +47,7 @@ public final class OnboardingValues extends SignalStoreValues {
public boolean hasOnboarding(@NonNull Context context) { public boolean hasOnboarding(@NonNull Context context) {
return shouldShowNewGroup() || return shouldShowNewGroup() ||
shouldShowInviteFriends() || shouldShowInviteFriends() ||
shouldShowSms(context) || shouldShowSms() ||
shouldShowAppearance() || shouldShowAppearance() ||
shouldShowAddPhoto(); shouldShowAddPhoto();
} }
@ -72,8 +72,8 @@ public final class OnboardingValues extends SignalStoreValues {
putBoolean(SHOW_SMS, value); putBoolean(SHOW_SMS, value);
} }
public boolean shouldShowSms(@NonNull Context context) { public boolean shouldShowSms() {
return getBoolean(SHOW_SMS, false) && !Util.isDefaultSmsProvider(context) && LocaleFeatureFlags.shouldSuggestSms(); return false;
} }
public void setShowAppearance(boolean value) { public void setShowAppearance(boolean value) {

View file

@ -148,7 +148,7 @@ public class OnboardingMegaphoneView extends FrameLayout {
data.add(TYPE_APPEARANCE); data.add(TYPE_APPEARANCE);
} }
if (SignalStore.onboarding().shouldShowSms(context)) { if (SignalStore.onboarding().shouldShowSms()) {
data.add(TYPE_SMS); data.add(TYPE_SMS);
} }