Merge pull request #178 from vtex-apps/change/conversion-trigger-state

Decision Manager automatic check
vtex-apps · Jul 30, 2024 · cacfb78 · cacfb78
2 parents 037ca81 + 1aad392
commit cacfb78
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,10 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
+### Changed
+- Changed trigger even for Decison Manager check to Pending Payment
+- Added logging
+
 ## [1.24.1] - 2024-07-23
 
 ### Fixed

diff --git a/dotnet/Services/VtexApiService.cs b/dotnet/Services/VtexApiService.cs
@@ -898,8 +898,9 @@ public async Task<bool> ProcessNotification(AllStatesNotification allStatesNotif
                             case CybersourceConstants.VtexOrderStatus.Invoiced:
                                 success = await this.ProcessInvoice(allStatesNotification.OrderId);
                                 break;
-                            case CybersourceConstants.VtexOrderStatus.ApprovePayment:
-                                _ = await this.ProcessConversions(2);
+                            case CybersourceConstants.VtexOrderStatus.PaymentPending:
+                                string result = await this.ProcessConversions(2);
+                                _context.Vtex.Logger.Info("ProcessConversions", null, result);
                                 break;
                         }