saulgoodman_best-practices

[shadow bridge]

👋 Welcome to your new thread!

⏲️ We'll be here soon! Typically we respond in a few minutes, but sometimes we might take a bit longer if the server is busy or if you have a particularly tricky question.

⏱️ We close idle threads, which makes them read-only. Once a thread is closed it won't be reopened, but you can always start a new thread if you have another question.

🔗 This thread will always be available, even after it's closed. You can find it again using Discord's search, or you can save this link: https://discord.com/channels/841573134531821608/1470335893821853788

📝 Have more to share? Add more details, code, screenshots, videos, etc. below.

[whole delta]

I don't see the updated field in paymentIntent as suggested by AI on docs

It doesn't show the type either in my application

[amber girder]

No there is no such a field

The only relibale way to check the latest status of the PaymentIntent is by doing a get by id via the public API

[whole delta]

how would you recommend I deal with this? for now I have logic setup like this

    db.update(ordersTable)
      .set({
        // status:
        status: stripeStatus as OrderStatus,
        paymentIntent,
      })
      .where(
        and(
          eq(ordersTable.id, orderId),
          notInArray(ordersTable.status, [
            OrderStatus.EXPIRED,
            OrderStatus.CANCELED,
            OrderStatus.COMPLETED,
          ]),
        ),
      );
  })

[amber girder]

Before making the db.update you need to check the PaymentIntent Id

[whole delta]

okay

Thank you 🙂

[amber girder]

Between, you shouldn't make heavy treatment in the webhook handlers, you should do it as async

[whole delta]

alright