TP-53724 | Hl x target correction (#9751) (#9752)

Co-authored-by: hitesh-kumar_navi <hitesh.kumar@navi.com>
Co-authored-by: Aman S <aman.s@navi.com>
Co-authored-by: Aman <amankasyapp@gmail.com>
Co-authored-by: Abhinav Gupta <abhinav.g@navi.com>
Co-authored-by: Shivam Goyal <shivam.goyal@navi.com>
Co-authored-by: Shaurya Rehan <shaurya.rehan@navi.com>
Co-authored-by: Saksham Mahajan <saksham.mahajan@navi.com>
Co-authored-by: shreyansu raj <shreyansu.raj@navi.com>
Co-authored-by: Kshitij Pramod Ghongadi <kshitij.pramod@navi.com>
Co-authored-by: Balrambhai Sharma <sharma.balrambhai@navi.com>
Co-authored-by: Apoorv Nigam <apoorv.nigam@navi.com>
Co-authored-by: Sayed Owais Ali <sayed.owais@navi.com>
Co-authored-by: Prajjaval Verma <prajjaval.verma@navi.com>
Co-authored-by: Ankit Yadav <ankit.yadav@navi.com>
Co-authored-by: nikhilkumar <nikhil.kumar@navi.com>
Co-authored-by: Aditya Piyush <aditya.piyush@navi.com>
Co-authored-by: kishan-kumar_navi <kishan.kumar@navi.com>

navi-common/src/main/java/com/navi/common/model/NetworkInfo.kt

@@ -31,5 +31,6 @@ enum class ModuleName {
     FEEDS,
     BBPS, // For Bharat Bill Payments System
     KYC_GATEWAY, // For KYC API
-    OPL // For OPL
+    OPL, // For OPL
+    HL
 }

navi_hl_dynamic/src/main/java/com/navi/hl/accountagreegator/repository/HLFinarkeinRepository.kt

@@ -9,7 +9,6 @@ package com.navi.hl.accountagreegator.repository
 
 import com.navi.common.account_aggregator.models.FinarkeinConsentRequest
 import com.navi.common.account_aggregator.models.FinarkeinEventRequest
-import com.navi.common.model.ModuleName
 import com.navi.common.network.retrofit.ResponseCallback
 import com.navi.hl.utils.retrofitService
 import com.navi.homeloan.common.models.accountaggregator.FinancialDataStatusRequest
@@ -17,28 +16,28 @@ import com.navi.homeloan.common.models.accountaggregator.FinancialDataStatusRequ
 class HLFinarkeinRepository : ResponseCallback() {
 
     suspend fun fetchFinarkeinAuthToken() =
-        apiResponseCallback(retrofitService().fetchFinarkeinAuthToken(target = ModuleName.LE.name))
+        apiResponseCallback(retrofitService().fetchFinarkeinAuthToken())
 
     suspend fun postFinarkeinEventUpdates(finarkeinEventRequest: FinarkeinEventRequest) =
         apiResponseCallback(
             retrofitService()
-                .postFinarkeinEventUpdates(target = ModuleName.LE.name, finarkeinEventRequest)
+                .postFinarkeinEventUpdates(finarkeinEventRequest)
         )
 
     suspend fun fetchFinarkeinConsentData(finarkeinConsentRequest: FinarkeinConsentRequest) =
         apiResponseCallback(
             retrofitService()
-                .fetchFinarkeinConsentData(target = ModuleName.LE.name, finarkeinConsentRequest)
+                .fetchFinarkeinConsentData(finarkeinConsentRequest)
         )
 
     suspend fun fetchFinarkeinPollingStatus(requestId: String) =
         apiResponseCallback(
-            retrofitService().fetchFinarkeinPollingStatus(target = ModuleName.LE.name, requestId)
+            retrofitService().fetchFinarkeinPollingStatus(requestId)
         )
 
     suspend fun fetchAccountAggregatorResponse(queryMap: HashMap<String, String>) =
         apiResponseCallback(
-            retrofitService().fetchAccountAggregatorResponse(target = ModuleName.LE.name, queryMap)
+            retrofitService().fetchAccountAggregatorResponse(queryMap)
         )
 
     suspend fun verifyFinancialDataStatusResponse(
@@ -48,7 +47,6 @@ class HLFinarkeinRepository : ResponseCallback() {
         apiResponseCallback(
             retrofitService()
                 .verifyFinancialDataStatusResponse(
-                    target = ModuleName.LE.name,
                     queryMap,
                     financialDataStatusRequest
                 )

navi_hl_dynamic/src/main/java/com/navi/hl/network/retrofit/HLRetrofitProvider.kt

@@ -34,7 +34,7 @@ class HLRetrofitProvider private constructor() {
                         baseUrl = getBaseUrl(),
                         appVersionName = BuildConfig.VERSION_NAME,
                         appVersionCode = BuildConfig.VERSION_CODE.toString(),
-                        moduleName = ModuleName.LE,
+                        moduleName = ModuleName.HL,
                         timeoutInSec = ApiConstants.API_CONNECT_TIMEOUT_VALUE
                     ),
                     context = NaviApplication.instance.applicationContext

navi_hl_dynamic/src/main/java/com/navi/hl/network/retrofit/RetrofitService.kt

@@ -189,12 +189,12 @@ interface RetrofitService {
     ): Response<GenericResponse<AggregatorSessionResponse>>
 
     @GET("/home-loan/customers/me/journey/v2/form")
-    suspend fun fetchAccountAggregatorResponse(
+    suspend fun fetchAccountAggregatorResponseForOneMoney(
         @QueryMap queryMap: HashMap<String, String>
     ): Response<GenericResponse<HomeLoanAccountAggregatorResponse>>
 
     @PATCH("/home-loan/customers/me/journey/v2/form")
-    suspend fun verifyFinancialDataStatusResponse(
+    suspend fun verifyFinancialDataStatusResponseForOneMoney(
         @QueryMap queryMap: HashMap<String, String>,
         @Body financialDataStatusRequest: FinancialDataStatusRequest
     ): Response<GenericResponse<UploadDataAsyncResponse>>
@@ -368,36 +368,30 @@ interface RetrofitService {
     // Finarkein Apis
     @GET("/financial-profile/account-aggregator/finarkein/session")
     suspend fun fetchFinarkeinAuthToken(
-        @Header("X-Target") target: String
     ): Response<GenericResponse<FinarkeinSessionResponse>>
 
     @POST("/financial-profile/account-aggregator/finarkein/inline-events")
     suspend fun postFinarkeinEventUpdates(
-        @Header("X-Target") target: String,
         @Body finarkeinEventRequest: FinarkeinEventRequest
     ): Response<GenericResponse<Boolean>>
 
     @POST("/financial-profile/account-aggregator/finarkein/consent")
     suspend fun fetchFinarkeinConsentData(
-        @Header("X-Target") target: String,
         @Body finarkeinConsentRequest: FinarkeinConsentRequest
     ): Response<GenericResponse<FinarkeinConsentResponse>>
 
     @GET("/requests/{requestId}")
     suspend fun fetchFinarkeinPollingStatus(
-        @Header("X-Target") target: String,
         @Path("requestId") requestId: String
     ): Response<GenericResponse<UploadDataAsyncResponse>>
 
     @GET("/home-loan/customers/me/journey/v2/form")
     suspend fun fetchAccountAggregatorResponse(
-        @Header("X-Target") target: String,
         @QueryMap queryMap: HashMap<String, String>
     ): Response<GenericResponse<HLAccountAggregatorResponse>>
 
     @PATCH("/home-loan/customers/me/journey/v2/form")
     suspend fun verifyFinancialDataStatusResponse(
-        @Header("X-Target") target: String,
         @QueryMap queryMap: HashMap<String, String>,
         @Body financialDataStatusRequest: FinancialDataStatusRequest
     ): Response<GenericResponse<UploadDataAsyncResponse>>

navi_hl_dynamic/src/main/java/com/navi/hl/steps/fragment/HomeLoanIncomeVerificationFragment.kt

@@ -204,7 +204,6 @@ class HomeLoanIncomeVerificationFragment :
         listener?.hideTrackerHeader()
         binding.footerView.setFooterInteractionListener(this)
         showLoader(isShimmerEffect = true)
-        checkForPerfiosRedirection()
         viewModel.fetchIncomeScreenResponse(queryMap)
         binding.searchContentRv.adapter = contentAdapter
     }
@@ -437,6 +436,7 @@ class HomeLoanIncomeVerificationFragment :
 
     private fun addObservers() {
         viewModel.screenResponse.observe(viewLifecycleOwner) {
+            it?.let { checkForPerfiosRedirection() }
             binding.response = it
             NaviTrackEvent.startScreen(screenName.plus(NaviHLAnalytics.POST_FIX_LANDED))
             it?.header?.run {

navi_hl_dynamic/src/main/java/com/navi/hl/steps/repository/HomeLoanAccountAggregatorRepository.kt

@@ -30,7 +30,7 @@ class HomeLoanAccountAggregatorRepository : ResponseCallback() {
         apiResponseCallback(retrofitService().fetchAsyncRequestDataWithErrorPayload(requestId))
 
     suspend fun fetchAccountAggregatorResponse(queryMap: HashMap<String, String>) =
-        apiResponseCallback(retrofitService().fetchAccountAggregatorResponse(queryMap))
+        apiResponseCallback(retrofitService().fetchAccountAggregatorResponseForOneMoney(queryMap))
 
     suspend fun verifyFinancialDataStatusResponse(
         queryMap: HashMap<String, String>,
@@ -38,6 +38,6 @@ class HomeLoanAccountAggregatorRepository : ResponseCallback() {
     ) =
         apiResponseCallback(
             retrofitService()
-                .verifyFinancialDataStatusResponse(queryMap, financialDataStatusRequest)
+                .verifyFinancialDataStatusResponseForOneMoney(queryMap, financialDataStatusRequest)
         )
 }