Merge pull request #136 from SocialGouv/fix/payment-completed-email

fix: check oldStatus of order to send sync email
SocialGouv · Nov 15, 2024 · f088cad · f088cad
2 parents c6e279d + 79336b6
commit f088cad
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/webapp/CHANGELOG.md b/webapp/CHANGELOG.md
@@ -12,6 +12,7 @@
 
 ## [0.64.1](https://github.com/SocialGouv/carte-jeune-engage/compare/v0.64.0...v0.64.1) (2024-11-15)
 
+
 ### Bug Fixes
 
 - margin between blocs home ([92214ff](https://github.com/SocialGouv/carte-jeune-engage/commit/92214ff61bc23c50549ace6ac968403193a65fbd))

diff --git a/webapp/src/server/api/routers/order.ts b/webapp/src/server/api/routers/order.ts
@@ -316,6 +316,7 @@ export const orderRouter = createTRPCRouter({
           newStatus !== order.status ||
           resultOrderStatusObject.etats_statut !== order.obiz_status
         ) {
+          const oldStatus = order.status;
           order = await ctx.payload.update({
             id: order_id,
             collection: "orders",
@@ -325,7 +326,7 @@ export const orderRouter = createTRPCRouter({
             },
           });
 
-          if (newStatus !== order.status && newStatus === "payment_completed") {
+          if (newStatus !== oldStatus && newStatus === "payment_completed") {
             const currentUser = await ctx.payload.findByID({
               collection: "users",
               id: order.user as number,
Original file line number	Diff line number	Diff line change
Expand Up		@@ -12,6 +12,7 @@

		## [0.64.1](https://github.com/SocialGouv/carte-jeune-engage/compare/v0.64.0...v0.64.1) (2024-11-15)


		### Bug Fixes

		- margin between blocs home ([92214ff](https://github.com/SocialGouv/carte-jeune-engage/commit/92214ff61bc23c50549ace6ac968403193a65fbd))
Expand Down