Merge branch 'master' into prod

AdguardTeam · Oct 2, 2019 · 173f7c0 · 173f7c0
2 parents d26cf58 + 2cd4e15
commit 173f7c0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/adguard_cb/build.gradle b/adguard_cb/build.gradle
@@ -18,8 +18,8 @@ android {
         minSdkVersion 21
         targetSdkVersion 28
         applicationId 'com.adguard.android.contentblocker'
-        versionCode 22000003
-        versionName '2.5.2'
+        versionCode 22000004
+        versionName '2.5.3'
         signingConfig signingConfigs.config
     }
 

diff --git a/adguard_cb/src/main/java/com/adguard/android/contentblocker/ui/MainActivity.java b/adguard_cb/src/main/java/com/adguard/android/contentblocker/ui/MainActivity.java
@@ -200,6 +200,9 @@ private void showRateDialog(Intent intent) {
             return;
         }
 
+        // We show notification MAX_RATE_DIALOG_COUNT times or while user didn't push a notification
+        preferencesService.setAppRated(true);
+
         int filledCount = intent.getIntExtra(STARS_COUNT, 0);
         final View dialogLayout = LayoutInflater.from(this).inflate(R.layout.rate_dialog, null);
         final ViewGroup starsLayout = dialogLayout.findViewById(R.id.stars_layout);
@@ -222,7 +225,6 @@ private void showRateDialog(Intent intent) {
         View.OnClickListener buttonsListener = v -> {
             if (v.getId() == R.id.button_submit) {
                 // TODO collect feedback
-                preferencesService.setAppRated(true);
                 showFeedbackSubmitDialog();
             }
             preferencesService.increaseRateAppDialogCount();