NTP-7179 | Narayan | Check balance experiment removal (#14641)

This commit is contained in:
Aditya Narayan Malik
2025-01-24 12:56:28 +05:30
committed by GitHub
parent efe07794b0
commit 431eea77ba
2 changed files with 0 additions and 29 deletions

View File

@@ -9,12 +9,10 @@ package com.navi.pay.common.usecase
import com.google.gson.reflect.TypeToken
import com.navi.base.utils.EMPTY
import com.navi.base.utils.orFalse
import com.navi.common.di.CoroutineDispatcherProvider
import com.navi.common.network.models.RepoResult
import com.navi.common.network.models.isSuccessWithData
import com.navi.common.resourcemanager.manager.ResourceManager.exceptionHandler
import com.navi.common.usecase.LitmusExperimentsUseCase
import com.navi.pay.common.model.config.NaviPayDefaultConfig
import com.navi.pay.common.model.view.CheckBalanceAnalyticsEventData
import com.navi.pay.common.model.view.CheckBalanceState
@@ -23,7 +21,6 @@ import com.navi.pay.onboarding.account.linked.model.network.CheckBalanceResponse
import com.navi.pay.utils.CHECK_BALANCE_ERROR_TRANSITION_TOTAL_DURATION
import com.navi.pay.utils.DEFAULT_CONFIG
import com.navi.pay.utils.INVALID_PIN
import com.navi.pay.utils.LITMUS_EXPERIMENT_NAVIPAY_CHECK_BALANCE_DURING_TRANSACTION
import com.navi.pay.utils.RUPEE_SYMBOL
import com.navi.pay.utils.getDisplayableAmount
import com.navi.pay.utils.isAccountIdOfTypeUpiLite
@@ -44,14 +41,11 @@ class AccountListCheckBalanceUseCase
constructor(
private val checkAccountBalanceUseCase: CheckAccountBalanceUseCase,
private val coroutineDispatcherProvider: CoroutineDispatcherProvider,
private val litmusExperimentsUseCase: LitmusExperimentsUseCase,
private val naviPayConfigUseCase: NaviPayConfigUseCase,
) {
val checkBalanceStateMap = mutableMapOf<String, MutableStateFlow<CheckBalanceState>>()
private val timerJobMap = mutableMapOf<String, Job?>()
private val isCheckBalanceExperimentEnabled = MutableStateFlow(false)
private var naviPayDefaultConfig = NaviPayDefaultConfig()
private val checkBalanceGenericError = "Unable to load"
@@ -61,7 +55,6 @@ constructor(
init {
updateNaviPayDefaultConfig()
setLitmusExperimentValues()
}
suspend fun onCheckBalanceClicked(
@@ -111,11 +104,6 @@ constructor(
val currentCheckBalanceState = checkBalanceStateMap[linkedAccountEntity.accountId]?.value
if (shouldIgnoreStateUpdate && currentCheckBalanceState != null) {
return
} else if (!isCheckBalanceExperimentEnabled.value) {
updateCheckBalanceStateInStateMapForAccountId(
accountId = linkedAccountEntity.accountId,
checkBalanceState = CheckBalanceState.None,
)
} else if (
isAccountEligible == false &&
(currentCheckBalanceState is CheckBalanceState.Success ||
@@ -177,20 +165,6 @@ constructor(
}
}
private fun setLitmusExperimentValues() {
coroutineScope.launch(coroutineDispatcherProvider.io) {
isCheckBalanceExperimentEnabled.update {
litmusExperimentsUseCase
.execute(
experimentName = LITMUS_EXPERIMENT_NAVIPAY_CHECK_BALANCE_DURING_TRANSACTION
)
?.variant
?.enabled
.orFalse()
}
}
}
private fun updateNaviPayDefaultConfig() {
coroutineScope.launch(coroutineDispatcherProvider.io) {
naviPayDefaultConfig =

View File

@@ -173,8 +173,6 @@ const val LITMUS_EXPERIMENT_NAVIPAY_LITE_AUTO_TOP_UP = "NaviPay-lite-auto-top-up
const val LITMUS_EXPERIMENT_NAVIPAY_LITE_DEFAULT_ENTERED_AMOUNT =
"NaviPay-lite-default-entered-amount"
const val LITMUS_EXPERIMENT_NAVIPAY_ORDER_TAG_SUMMARY = "NaviPay-order-tag-summary"
const val LITMUS_EXPERIMENT_NAVIPAY_CHECK_BALANCE_DURING_TRANSACTION =
"NaviPay-exp-check-balance-during-transaction"
const val LITMUS_EXPERIMENT_NAVIPAY_FREQUENT_CONTACT_IN_QR_SCANNER =
"NaviPay-frequent-contact-in-qr-scanner"
const val LITMUS_EXPERIMENT_NAVIPAY_SMV_BINDING = "NaviPay-exp-smv-binding"
@@ -184,7 +182,6 @@ val NAVI_PAY_LITMUS_EXPERIMENTS =
LITMUS_EXPERIMENT_NAVIPAY_LITE_AUTO_TOP_UP,
LITMUS_EXPERIMENT_NAVIPAY_LITE_DEFAULT_ENTERED_AMOUNT,
LITMUS_EXPERIMENT_NAVIPAY_ORDER_TAG_SUMMARY,
LITMUS_EXPERIMENT_NAVIPAY_CHECK_BALANCE_DURING_TRANSACTION,
LITMUS_EXPERIMENT_NAVIPAY_FREQUENT_CONTACT_IN_QR_SCANNER,
LITMUS_EXPERIMENT_NAVIPAY_SMV_BINDING,
LITMUS_EXPERIMENT_NAVI_REPUBLIC_DAY,