Merge pull request #12795 from woocommerce/issue/12469-blaze-objectiv…

…e-store-switch

[Blaze i4] Add switch to the objective screen

WooCommerce/src/main/kotlin/com/woocommerce/android/AppPrefs.kt

@@ -127,7 +127,8 @@ object AppPrefs {
         TIMES_AI_PRODUCT_CREATION_SURVEY_DISPLAYED,
         AI_PRODUCT_CREATION_SURVEY_DISMISSED,
         CUSTOM_FIELDS_TOP_BANNER_DISMISSED,
-        BLAZE_CAMPAIGN_SELECTED_OBJECTIVE
+        BLAZE_CAMPAIGN_SELECTED_OBJECTIVE,
+        BLAZE_CAMPAIGN_OBJECTIVE_SWITCH_CHECKED
     }
 
     /**
@@ -277,6 +278,10 @@ object AppPrefs {
         get() = getString(DeletablePrefKey.BLAZE_CAMPAIGN_SELECTED_OBJECTIVE, "")
         set(value) = setString(DeletablePrefKey.BLAZE_CAMPAIGN_SELECTED_OBJECTIVE, value)
 
+    var blazeCampaignObjectiveSwitchChecked: Boolean
+        get() = getBoolean(DeletablePrefKey.BLAZE_CAMPAIGN_OBJECTIVE_SWITCH_CHECKED, true)
+        set(value) = setBoolean(DeletablePrefKey.BLAZE_CAMPAIGN_OBJECTIVE_SWITCH_CHECKED, value)
+
     fun getProductSortingChoice(currentSiteId: Int) = getString(getProductSortingKey(currentSiteId)).orNullIfEmpty()
 
     fun setProductSortingChoice(currentSiteId: Int, value: String) {

WooCommerce/src/main/kotlin/com/woocommerce/android/AppPrefsWrapper.kt

@@ -38,6 +38,8 @@ class AppPrefsWrapper @Inject constructor() {
 
     var blazeCampaignSelectedObjective by AppPrefs::blazeCampaignSelectedObjective
 
+    var blazeCampaignObjectiveSwitchChecked by AppPrefs::blazeCampaignObjectiveSwitchChecked
+
     fun getAppInstallationDate() = AppPrefs.installationDate
 
     fun getReceiptUrl(localSiteId: Int, remoteSiteId: Long, selfHostedSiteId: Long, orderId: Long) =

WooCommerce/src/main/kotlin/com/woocommerce/android/analytics/AnalyticsEvent.kt

@@ -1031,6 +1031,7 @@ enum class AnalyticsEvent(override val siteless: Boolean = false) : IAnalyticsEv
     BLAZE_CREATION_EDIT_LOCATION_SAVE_TAPPED,
     BLAZE_CREATION_EDIT_DESTINATION_SAVE_TAPPED,
     BLAZE_CAMPAIGN_CREATION_FEEDBACK,
+    BLAZE_CAMPAIGN_OBJECTIVE_SAVED,
 
     // Hazmat Shipping Declaration
     CONTAINS_HAZMAT_CHECKED,

WooCommerce/src/main/kotlin/com/woocommerce/android/analytics/AnalyticsTracker.kt

@@ -618,6 +618,7 @@ class AnalyticsTracker private constructor(
         const val KEY_BLAZE_IS_AI_CONTENT = "is_ai_suggested_ad_content"
         const val KEY_BLAZE_ERROR = "blaze_creation_error"
         const val KEY_BLAZE_CAMPAIGN_TYPE = "campaign_type"
+        const val KEY_BLAZE_OBJECTIVE = "objective"
 
         const val PRODUCT_TYPES = "product_types"
         const val HAS_ADDONS = "has_addons"

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/blaze/BlazeRepository.kt

@@ -379,6 +379,16 @@ class BlazeRepository @Inject constructor(
         }
     }
 
+    fun isCampaignObjectiveSwitchChecked() = appPrefsWrapper.blazeCampaignObjectiveSwitchChecked
+
+    fun setCampaignObjectiveSwitchChecked(enabled: Boolean) {
+        appPrefsWrapper.blazeCampaignObjectiveSwitchChecked = enabled
+    }
+
+    fun storeSelectedObjective(objectiveId: String) {
+        appPrefsWrapper.blazeCampaignSelectedObjective = objectiveId
+    }
+
     @Parcelize
     data class CampaignDetails(
         val productId: Long,

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/blaze/creation/objective/BlazeCampaignObjectiveScreen.kt

@@ -15,6 +15,7 @@ import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.lazy.LazyColumn
 import androidx.compose.foundation.lazy.items
 import androidx.compose.foundation.shape.RoundedCornerShape
+import androidx.compose.material.Divider
 import androidx.compose.material.MaterialTheme
 import androidx.compose.material.Scaffold
 import androidx.compose.material.Text
@@ -34,7 +35,9 @@ import com.woocommerce.android.R
 import com.woocommerce.android.ui.blaze.creation.objective.BlazeCampaignObjectiveViewModel.ObjectiveItem
 import com.woocommerce.android.ui.blaze.creation.objective.BlazeCampaignObjectiveViewModel.ObjectiveViewState
 import com.woocommerce.android.ui.compose.annotatedStringRes
+import com.woocommerce.android.ui.compose.component.BottomSheetSwitchColors
 import com.woocommerce.android.ui.compose.component.Toolbar
+import com.woocommerce.android.ui.compose.component.WCSwitch
 import com.woocommerce.android.ui.compose.component.WCTextButton
 import com.woocommerce.android.ui.compose.preview.LightDarkThemePreviews
 
@@ -45,7 +48,8 @@ fun BlazeCampaignObjectiveScreen(viewModel: BlazeCampaignObjectiveViewModel) {
             state = state,
             onBackPressed = viewModel::onBackPressed,
             onSaveTapped = viewModel::onSaveTapped,
-            onObjectiveTapped = viewModel::onItemToggled
+            onObjectiveTapped = viewModel::onItemToggled,
+            onStoreObjectiveSwitchChanged = viewModel::onStoreObjectiveSwitchChanged
         )
     }
 }
@@ -56,6 +60,7 @@ private fun ObjectiveScreen(
     onBackPressed: () -> Unit,
     onSaveTapped: () -> Unit,
     onObjectiveTapped: (ObjectiveItem) -> Unit,
+    onStoreObjectiveSwitchChanged: (Boolean) -> Unit
 ) {
     Scaffold(
         topBar = {
@@ -81,7 +86,9 @@ private fun ObjectiveScreen(
                 .fillMaxSize()
         ) {
             LazyColumn(
-                modifier = Modifier.padding(vertical = 4.dp)
+                modifier = Modifier
+                    .padding(vertical = 4.dp)
+                    .weight(1f)
             ) {
                 items(state.items) {
                     ObjectiveListItem(
@@ -99,6 +106,16 @@ private fun ObjectiveScreen(
                     )
                 }
             }
+            Divider()
+            WCSwitch(
+                text = stringResource(id = R.string.blaze_campaign_objective_save_selection_switch_label),
+                checked = state.isStoreSelectionButtonToggled,
+                onCheckedChange = { onStoreObjectiveSwitchChanged(it) },
+                modifier = Modifier
+                    .fillMaxWidth()
+                    .padding(16.dp),
+                colors = BottomSheetSwitchColors()
+            )
         }
     }
 }
@@ -216,5 +233,6 @@ fun PreviewObjectiveScreen() {
         onBackPressed = { },
         onSaveTapped = { },
         onObjectiveTapped = { },
+        onStoreObjectiveSwitchChanged = { }
     )
 }

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/blaze/creation/objective/BlazeCampaignObjectiveViewModel.kt

@@ -4,11 +4,15 @@ import android.os.Parcelable
 import androidx.lifecycle.SavedStateHandle
 import androidx.lifecycle.asLiveData
 import androidx.lifecycle.viewModelScope
+import com.woocommerce.android.analytics.AnalyticsEvent.BLAZE_CAMPAIGN_OBJECTIVE_SAVED
+import com.woocommerce.android.analytics.AnalyticsTracker
+import com.woocommerce.android.analytics.AnalyticsTrackerWrapper
 import com.woocommerce.android.ui.blaze.BlazeRepository
 import com.woocommerce.android.viewmodel.MultiLiveEvent.Event.Exit
 import com.woocommerce.android.viewmodel.MultiLiveEvent.Event.ExitWithResult
 import com.woocommerce.android.viewmodel.ScopedViewModel
 import com.woocommerce.android.viewmodel.getNullableStateFlow
+import com.woocommerce.android.viewmodel.getStateFlow
 import com.woocommerce.android.viewmodel.navArgs
 import dagger.hilt.android.lifecycle.HiltViewModel
 import kotlinx.coroutines.flow.Flow
@@ -20,15 +24,22 @@ import javax.inject.Inject
 
 @HiltViewModel
 class BlazeCampaignObjectiveViewModel @Inject constructor(
-    blazeRepository: BlazeRepository,
-    savedStateHandle: SavedStateHandle
+    private val blazeRepository: BlazeRepository,
+    savedStateHandle: SavedStateHandle,
+    private val analyticsTrackerWrapper: AnalyticsTrackerWrapper
 ) : ScopedViewModel(savedStateHandle) {
     private val navArgs: BlazeCampaignObjectiveFragmentArgs by savedStateHandle.navArgs()
 
     private val selectedId = savedStateHandle.getNullableStateFlow(
         scope = viewModelScope,
         initialValue = navArgs.selectedObjectiveId,
-        clazz = String::class.java
+        clazz = String::class.java,
+        key = "selectedId"
+    )
+    private val storeObjectiveSwitchState = savedState.getStateFlow(
+        scope = this,
+        initialValue = blazeRepository.isCampaignObjectiveSwitchChecked(),
+        key = "storeObjectiveSwitchState"
     )
 
     private val items: Flow<List<ObjectiveItem>> =
@@ -38,8 +49,12 @@ class BlazeCampaignObjectiveViewModel @Inject constructor(
             }
         }
 
-    val viewState = combine(items, selectedId) { items, selectedId ->
-        ObjectiveViewState(items = items, selectedItemId = selectedId)
+    val viewState = combine(
+        items,
+        selectedId,
+        storeObjectiveSwitchState
+    ) { items, selectedId, storeObjectiveSwitchState ->
+        ObjectiveViewState(items, selectedId, storeObjectiveSwitchState)
     }.asLiveData()
 
     fun onItemToggled(item: ObjectiveItem) {
@@ -50,15 +65,30 @@ class BlazeCampaignObjectiveViewModel @Inject constructor(
         triggerEvent(Exit)
     }
 
+    fun onStoreObjectiveSwitchChanged(checked: Boolean) {
+        storeObjectiveSwitchState.update { checked }
+    }
+
     fun onSaveTapped() {
-        selectedId.value?.let { triggerEvent(ExitWithResult(ObjectiveResult(it))) }
-        // TODO analyticsTrackerWrapper.track(BLAZE_...)
+        viewState.value?.isStoreSelectionButtonToggled?.let {
+            blazeRepository.setCampaignObjectiveSwitchChecked(it)
+            if (it) {
+                blazeRepository.storeSelectedObjective(selectedId.value.orEmpty())
+            }
+        }
+        selectedId.value?.let {
+            triggerEvent(ExitWithResult(ObjectiveResult(it)))
+            analyticsTrackerWrapper.track(
+                stat = BLAZE_CAMPAIGN_OBJECTIVE_SAVED,
+                properties = mapOf(AnalyticsTracker.KEY_BLAZE_OBJECTIVE to it)
+            )
+        }
     }
 
     data class ObjectiveViewState(
         val items: List<ObjectiveItem>,
         val selectedItemId: String? = null,
-        val isStoreSelectionButtonToggled: Boolean = false,
+        val isStoreSelectionButtonToggled: Boolean = true,
     ) {
         val isSaveButtonEnabled: Boolean
             get() = !selectedItemId.isNullOrEmpty()

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/blaze/creation/preview/BlazeCampaignCreationPreviewViewModel.kt

@@ -182,7 +182,8 @@ class BlazeCampaignCreationPreviewViewModel @Inject constructor(
                         AnalyticsTracker.VALUE_EVERGREEN_CAMPAIGN
 
                     else -> AnalyticsTracker.VALUE_START_END_CAMPAIGN
-                }
+                },
+                AnalyticsTracker.KEY_BLAZE_OBJECTIVE to campaignDetails.value?.objectiveId,
             )
         )
         campaignDetails.value?.let {

WooCommerce/src/main/res/values/strings.xml

@@ -3917,6 +3917,14 @@
     <string name="blaze_campaign_preview_missing_content_dialog_positive_button">Add</string>
     <string name="blaze_campaign_preview_missing_objective_dialog_text">Please select an objective for the Blaze campaign</string>
     <string name="blaze_campaign_preview_missing_objective_dialog_positive_button">Select objective</string>
+
+    <!--
+    Blaze campaign objective screen
+    -->
+    <string name="blaze_campaign_objective_select_objective_label">Select objective %s</string>
+    <string name="blaze_campaign_objective_good_for"><![CDATA[<b>Good for:</b> %s]]></string>
+    <string name="blaze_campaign_objective_save_selection_switch_label">Save my selection for future campaigns</string>
+
     <!--
     Blaze campaign budget screen
     -->
@@ -3992,8 +4000,6 @@
     <string name="blaze_campaign_creation_location_search_message">Start typing country, state or city to see available options</string>
     <string name="blaze_campaign_creation_location_search_no_results_message">No location found.\nPlease try again.</string>
     <string name="blaze_campaign_creation_location_search_failed_message">Searching failed.\nPlease try again</string>
-    <string name="blaze_campaign_objective_select_objective_label">Select objective %s</string>
-    <string name="blaze_campaign_objective_good_for"><![CDATA[<b>Good for:</b> %s]]></string>
 
     <!--
     Blaze edit ad destination