Skip to content

Commit 6f4e95f

Browse files
committed
Show a confirmation dialog on registration and verification success
Signed-off-by: Arka Prava Basu <[email protected]>
1 parent 35c968f commit 6f4e95f

File tree

2 files changed

+38
-3
lines changed

2 files changed

+38
-3
lines changed

src/main/java/org/havenapp/main/SettingsFragment.java

Lines changed: 33 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
import android.app.Activity;
1010
import android.app.ProgressDialog;
1111
import android.content.Context;
12+
import android.content.DialogInterface;
1213
import android.content.Intent;
1314
import android.content.SharedPreferences;
1415
import android.content.pm.PackageManager;
@@ -38,6 +39,7 @@
3839
import java.util.Locale;
3940

4041
import androidx.annotation.NonNull;
42+
import androidx.appcompat.app.AlertDialog;
4143
import androidx.appcompat.app.AppCompatActivity;
4244
import androidx.core.app.ActivityCompat;
4345
import androidx.core.content.ContextCompat;
@@ -420,7 +422,6 @@ public void onSharedPreferenceChanged(SharedPreferences sharedPreferences, Strin
420422
}
421423
activateSignal(preferences.getSignalUsername(), text);
422424
onRemoteNotificationParameterChange();
423-
checkSignalUsernameVerification();
424425
break;
425426
}
426427
case PreferenceManager.REMOTE_PHONE_NUMBER:
@@ -615,7 +616,7 @@ public void onSuccess(@NonNull String msg) {
615616
if (isAdded() && getActivity() != null) {
616617
progressDialog.dismiss();
617618
}
618-
Toast.makeText(getContext(), msg, Toast.LENGTH_SHORT).show();
619+
showRegistrationSuccessDialog();
619620
}
620621

621622
@Override
@@ -637,7 +638,8 @@ public void onSuccess(@NonNull String msg) {
637638
}
638639
// mark that the current registered signal username is verified
639640
preferences.setVerifiedSignalUsername(preferences.getSignalUsername());
640-
Toast.makeText(getContext(), msg, Toast.LENGTH_SHORT).show();
641+
checkSignalUsernameVerification();
642+
showVerificationSuccessDialog();
641643
}
642644

643645
@Override
@@ -651,6 +653,34 @@ public void onFailure(@NonNull String msg) {
651653
}
652654
}
653655

656+
private void showRegistrationSuccessDialog() {
657+
if (!isAdded() || getActivity() == null) {
658+
return;
659+
}
660+
661+
new AlertDialog.Builder(getActivity())
662+
.setTitle(R.string.registration_successful)
663+
.setMessage(getString(R.string.signal_reg_success_desc, preferences.getSignalUsername()))
664+
.setPositiveButton(R.string.verify, (dialog, which) -> {
665+
dialog.dismiss();
666+
findPreference(PreferenceManager.VERIFY_SIGNAL).performClick();
667+
})
668+
.setNegativeButton(R.string.ok, (dialog, which) -> dialog.dismiss())
669+
.show();
670+
}
671+
672+
private void showVerificationSuccessDialog() {
673+
if (!isAdded() || getActivity() == null) {
674+
return;
675+
}
676+
677+
new AlertDialog.Builder(getActivity())
678+
.setTitle(R.string.verification_successful)
679+
.setMessage(R.string.signal_verification_success_desc)
680+
.setPositiveButton(R.string.ok, (dialog, which) -> dialog.dismiss())
681+
.show();
682+
}
683+
654684
private void resetSignal(String username) {
655685
if (checkValidString((username))) {
656686
SignalSender sender = SignalSender.getInstance(mActivity, username.trim());

src/main/res/values/strings.xml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -182,5 +182,10 @@
182182
<string name="signal_registration_desc">Please wait while we register you to Signal services</string>
183183
<string name="verifying_signal">Verifying</string>
184184
<string name="verifying_signal_desc">Please wait while we verify your registration to Signal services</string>
185+
<string name="ok">Ok</string>
186+
<string name="registration_successful">Registration Successful</string>
187+
<string name="signal_reg_success_desc">You have successfully registered to Signal services. Next Step is to verify the registration for %s</string>
188+
<string name="verification_successful">Verification Successful</string>
189+
<string name="signal_verification_success_desc">All set to send Haven alerts from Signal!</string>
185190

186191
</resources>

0 commit comments

Comments
 (0)