shouldBatchDisplayPropositions state moved to viewmodel

siddique-adobe · Oct 30, 2024 · cd7f86b · cd7f86b
1 parent d939c44
commit cd7f86b
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 12 deletions.
diff --git a/code/testapp/src/main/java/com/adobe/marketing/optimizeapp/OffersScreen.kt b/code/testapp/src/main/java/com/adobe/marketing/optimizeapp/OffersScreen.kt
@@ -26,8 +26,6 @@ import androidx.compose.material.Switch
 import androidx.compose.material.Text
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.LaunchedEffect
-import androidx.compose.runtime.mutableStateOf
-import androidx.compose.runtime.remember
 import androidx.compose.runtime.snapshotFlow
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
@@ -65,7 +63,6 @@ private val displayPropositionsHandler: (List<Offer>) -> Unit = { offers ->
 
 @Composable
 fun OffersView(viewModel: MainViewModel) {
-    val shouldBatchDisplayPropositions = remember { mutableStateOf(false) }
     var listState = rememberLazyListState()
     Column(
         modifier = Modifier
@@ -81,15 +78,15 @@ fun OffersView(viewModel: MainViewModel) {
             ) {
 
                 OffersSectionText(sectionName = "Text Offers")
-                TextOffers(shouldCallDisplayProposition = !shouldBatchDisplayPropositions.value)
+                TextOffers(shouldCallDisplayProposition = !viewModel.shouldBatchDisplayPropositions)
                 OffersSectionText(sectionName = "Image Offers")
                 ImageOffers()
                 OffersSectionText(sectionName = "HTML Offers")
                 HTMLOffers()
                 OffersSectionText(sectionName = "JSON Offers")
-                JSONOffers(shouldCallDisplayProposition = !shouldBatchDisplayPropositions.value)
+                JSONOffers(shouldCallDisplayProposition = !viewModel.shouldBatchDisplayPropositions)
                 OffersSectionText(sectionName = "Target Offers")
-                TargetOffersView(shouldCallDisplayProposition = !shouldBatchDisplayPropositions.value)
+                TargetOffersView(shouldCallDisplayProposition = !viewModel.shouldBatchDisplayPropositions)
             }
         } else {
             LazyColumn(
@@ -108,7 +105,7 @@ fun OffersView(viewModel: MainViewModel) {
                                 TextOffers(
                                     offers = viewModel.optimizePropositionStateMap[viewModel.textDecisionScope?.name]?.offers,
                                     listState = listState,
-                                    shouldCallDisplayProposition = !shouldBatchDisplayPropositions.value
+                                    shouldCallDisplayProposition = !viewModel.shouldBatchDisplayPropositions
                                 )
                             }
 
@@ -133,7 +130,7 @@ fun OffersView(viewModel: MainViewModel) {
                                 JSONOffers(
                                     offers = viewModel.optimizePropositionStateMap[viewModel.jsonDecisionScope?.name]?.offers,
                                     listState = listState,
-                                    shouldCallDisplayProposition = !shouldBatchDisplayPropositions.value
+                                    shouldCallDisplayProposition = !viewModel.shouldBatchDisplayPropositions
                                 )
                             }
 
@@ -142,13 +139,13 @@ fun OffersView(viewModel: MainViewModel) {
                                 TargetOffersView(
                                     offers = viewModel.optimizePropositionStateMap[viewModel.targetMboxDecisionScope?.name]?.offers,
                                     listState = listState,
-                                    shouldCallDisplayProposition = !shouldBatchDisplayPropositions.value
+                                    shouldCallDisplayProposition = !viewModel.shouldBatchDisplayPropositions
                                 )
                             }
                         }
                     })
             }.also {
-                if (shouldBatchDisplayPropositions.value) {
+                if (viewModel.shouldBatchDisplayPropositions) {
                     displayPropositionsHandler(viewModel.optimizePropositionStateMap.values.toList())
                 }
             }
@@ -178,8 +175,8 @@ fun OffersView(viewModel: MainViewModel) {
             )
 
             Switch(
-                checked = shouldBatchDisplayPropositions.value,
-                onCheckedChange = { shouldBatchDisplayPropositions.value = it })
+                checked = viewModel.shouldBatchDisplayPropositions,
+                onCheckedChange = { viewModel.shouldBatchDisplayPropositions = it })
         }
         Spacer(
             modifier = Modifier

diff --git a/code/testapp/src/main/java/com/adobe/marketing/optimizeapp/viewmodels/MainViewModel.kt b/code/testapp/src/main/java/com/adobe/marketing/optimizeapp/viewmodels/MainViewModel.kt
@@ -44,6 +44,8 @@ class MainViewModel: ViewModel() {
 
     var optimizePropositionStateMap = mutableStateMapOf<String, OptimizeProposition>()
 
+    var shouldBatchDisplayPropositions by mutableStateOf(false)
+
     private val optimizePropositionUpdateCallback = object : AdobeCallbackWithError<Map<DecisionScope, OptimizeProposition>> {
         override fun call(propositions: Map<DecisionScope, OptimizeProposition>?) {
             propositions?.forEach {