hyperswitch
feat(process_tracker): Invoke record back flow in PCR workflow [V2]
#7660
Merged
Merged
feat(process_tracker): Invoke record back flow in PCR workflow [V2] #7660
likhinbopanna
merged
7 commits into
main
from
record_back_flow_in_pt
invoke record back flow in pcr workflow
46d730ba
semanticdiff-com16 days ago (edited 16 days ago)
Review changes with
Changed Files
| File | Status |
|---|---|
 crates/hyperswitch_interfaces/src/connector_integration_v2.rs |
 56% smaller |
| crates/router/src/core/revenue_recovery.rs |
 42% smaller |
| crates/hyperswitch_domain_models/src/payments.rs |
 37% smaller |
| crates/router/src/core/revenue_recovery/types.rs |
 18% smaller |
| crates/diesel_models/src/types.rs |
 13% smaller |
| crates/router/src/workflows/revenue_recovery.rs |
13% smaller |
| crates/storage_impl/src/errors.rs |
 0% smaller |
Merge branch 'main' into record_back_flow_in_pt
ffb1c239
typos fix
654e3e98
Merge branch 'main' into record_back_flow_in_pt
2a0d7f50
srujanchikke
commented
on 2025-03-28
crates/router/src/workflows/revenue_recovery.rs
| 88 | storage_errors::RecoveryError::BillingMerchantConnectorAccountIdNotFound.into(), | ||
| 89 | ))?; | ||
| 90 | |||
| 91 | let billing_mca = store | ||
| 92 | .find_merchant_connector_account_by_id( | ||
| 93 | key_manager_state, | ||
| 94 | &billing_merchant_connector_account_id, | ||
| 95 | &pcr_data.key_store, | ||
| 96 | ) | ||
| 97 | .await?; |
srujanchikke16 days agoNote : We cannot avoid db call here since we need connector creds to make record back. Get MCA will not give api keys to call connector.
jarnura
requested changes
on 2025-03-29
crates/hyperswitch_interfaces/src/connector_integration_v2.rs
| 50 | 51 | + api::ConnectorMandateRevokeV2 | |
| 51 | 52 | + api::authentication_v2::ExternalAuthenticationV2 | |
| 52 | + api::UnifiedAuthenticationServiceV2, | ||
| 53 | + api::UnifiedAuthenticationServiceV2 | ||
| 54 | + api::revenue_recovery_v2::RevenueRecoveryV2, |
jarnura15 days ago
Suggested change
| + api::revenue_recovery_v2::RevenueRecoveryV2, | |
| + api::revenue_recovery::RevenueRecovery, |
jarnura12 days ago👍 1Take this up in next pr
jarnura
dismissed these changes
on 2025-04-01
Merge branch 'main' into record_back_flow_in_pt
195698d6
Aprabhat19
dismissed these changes
on 2025-04-01
crates/router/src/core/revenue_recovery/types.rs
| 200 | billing_mca: &merchant_connector_account::MerchantConnectorAccount, | ||
| 176 | 201 | ) -> Result<(), errors::ProcessTrackerError> { | |
| 177 | let db = &*state.store; | ||
| 202 | let db: &dyn StorageInterface = &*state.store; |
Aprabhat1912 days ago (edited 12 days ago)nit: Can you remove this
resolve comments
ecf94e37
jarnura
approved these changes
on 2025-04-01
Aprabhat19
approved these changes
on 2025-04-01
Merge branch 'main' into record_back_flow_in_pt
ad8f19d7
Login to write a write a comment.
Reviewers
Assignees
Labels
A-core
A-process-tracker
api-v2
Milestone
No milestone
Type of Change
Description
What is record back flow ?
After payment is retried by revenue recovery by PCR workflow, We need to update the billing processor(update the invoce or record the attempt).This PR invokes record back functionality in process tracker flow for Terminal status.
Note: Adding support for retrying failed outgoing recordback webhooks is still pending attached in this issues.
Additional Changes
Motivation and Context
How did you test it?
Can be tested once this PR is merged #7614
Checklist
cargo +nightly fmt --allcargo clippy