checkPurchaseOrderStatus y correo de compra completada en 9punto5 #269

TextC0de · 2024-09-12T07:49:15Z

Este PR implementa el envío de correos electrónicos de confirmación de compra específicos para tickets de 9punto5.

Los cambios principales incluyen:

Creación de una nueva plantilla de correo electrónico PurchaseOrderSuccessful9punto5 para 9punto5.
Modificación del servicio de correo transaccional para utilizar la nueva plantilla cuando se trata de la comunidad "9punto5".
Actualización de la lógica de sincronización del estado de pago de órdenes de compra para enviar el correo de confirmación cuando el pago se marca como completado.
Adición de pruebas unitarias para validar el nuevo flujo de envío de correos.

emails/templates/helpers/fonts.tsx

github-actions · 2024-09-12T07:51:25Z

Coverage Report

Status	Category	Percentage	Covered / Total
🔵	Lines	81.73%	14197 / 17369
🔵	Statements	81.73%	14197 / 17369
🔵	Functions	77.07%	390 / 506
🔵	Branches	78.97%	1022 / 1294

File Coverage

File	Stmts	% Branch	% Funcs	% Lines	Uncovered Lines
Changed Files
src/authz/helpers.ts	83.33%	87.5%	85.71%	83.33%	89-107
src/datasources/db/purchaseOrders.ts	100%	100%	100%	100%
src/generated/types.ts	100%	100%	100%	100%
src/schema/purchaseOrder/actions.tsx	42.56%	60%	55.55%	42.56%	30-51, 97-102, 107-112, 143-234, 237-305, 308-546, 616-617, 653-654, 661-664, 711-712
src/schema/purchaseOrder/mutations.tsx	71.87%	83.33%	80%	71.87%	36-76, 118-119, 149-150
src/schema/purchaseOrder/tests/checkPurchaseOrderStatus.generated.ts	100%	100%	100%	100%
src/schema/userTickets/mutations.ts	84.93%	67.36%	94.73%	84.93%	99-100, 146-147, 169-170, 211-212, 223-224, 235-236, 283-284, 295-300, 312-317, 320-325, 374-379, 394-399, 446-447, 457-458, 478-481, 493-494, 500-527, 538, 561-562, 583-584, 630-635, 638-643
workers/purchase_order_payment_sync_cron/scheduled.tsx	60.24%	50%	33.33%	60.24%	50-54, 56-83

Generated in workflow #1177

…ync_cron

Signed-off-by: Ignacio Guzmán <[email protected]>

emails/templates/tickets/purchase-order-successful/9punto5.tsx

emails/templates/tickets/purchase-order-successful/jschile.tsx

fforres · 2024-09-13T18:49:43Z

src/authz/helpers.ts

@@ -35,6 +35,9 @@ const isOwnerOfPurchaseOrder = async ({
  const isOwner = await DB.query.purchaseOrdersSchema.findFirst({
    where: (po, { eq, and }) =>
      and(eq(po.id, purchaseOrderId), eq(po.userId, user.id)),
+    columns: {


Es esto necesario si estamos devolviendo un Boolean(isOwner)

En este caso la tabla no tiene muchas columnas ni hacemos joins pero creo creo que seleccionar solo las columnas necesarias es una buena práctica.
¿Es necesario? No

🤘🏼 me parece overkill, pero no estoy en contra

fforres · 2024-09-13T18:52:13Z

src/schema/purchaseOrder/actions.tsx

-  email,
-}: {
-  DB: ORM_TYPE;
+type SendPurchaseOrderSuccessfulEmailArgs = {


porque este cambio? la idea inicial es no tener que obtener toda la data de cada purchase order previamente cuando quieras usar este metodo.

Creo que hay optimizar la aplicación para evitar hacer consultas repetidas.

Esta función era utilizada en createPaymentIntent si todos los tickets son gratis.

En createPaymentIntent se está utilizando la función "fetchPurchaseOrderInformation" que ya traía la información de los ticketTemplate, prices y currencies.

En esta función se estaba repitiendo la misma consulta.

Por eso en principio había hecho este cambio, para evitar consultas duplicadas a la BD.

Sacar la informacion de la BDD tiene q ser... 150ms? al ojo? Por ahora, no buscaría optimizar llamadas o performance (aka, evitar hacer consultas, etc) sino, hacer la experiencia de desarrollo mas facil por ahora.

Si esta, dale dejemoslo, pero 300ms no van a causar un drop de uso.

src/schema/purchaseOrder/actions.tsx

fforres · 2024-09-13T19:03:52Z

src/schema/purchaseOrder/actions.tsx

@@ -419,14 +413,6 @@ export const createPaymentIntent = async ({

    const userTicketsIds = userTickets.map((t) => t.id);

-    await sendConfirmationEmail({


Si todos los tickets son gratis, esto debería generar una PO "lista"....

Estamos validando y enviando un correo en este caso en otro lado?

Tienes razón, mala mia. Pasé por alto el "if (allTicketsAreFree)"
Me fijaré los tests de esto.

¿En que casos, uno le pasaría generatePaymentLink como null a la mutación "claimUserTicket"?
El caso que se me ocurre es cuando todos los tickets son gratuitos.

Si se le pasa "generatePaymentLink" como null, entonces "createPaymentIntent" nunca es llamado y el correo no va a ser enviado.

gql_api/src/schema/userTickets/mutations.ts

Line 257 in 7fd1471

builder.mutationField("claimUserTicket", (t) =>

joseglego · 2024-09-13T21:42:37Z

emails/templates/helpers/fonts.tsx

@@ -237,27 +237,200 @@ export const PoppinsFont = () => (
  </>
 );

+const roboto = [


Tengo 2 preguntas:

Son necesarias todas estás fuentes? Quizás solo incluir las que necesitas

Esto descarga todas las fuentes así no las use?

Esto descarga todas las fuentes así no las use?

Si estan en el HTML final, deberian

Seguí el estilo de lo mismo que se está haciendo con poppins, en la Poppins se está haciendo lo mismo de cargar un montón de fuentes.

Entiendo que sí una font-face no es utilizada no se descarga:

https://jaketrent.com/post/when-font-face-is-downloaded/

Igual estuve averiguando y vi que Gmail no soporta custom web fonts:

https://stackoverflow.com/a/70455614

https://www.caniemail.com/features/css-at-font-face/

https://react.email/docs/components/font

Lo recomendable es usar web fonts que están disponibles en la mayoría de clientes de correo.

Hice unos tests en Gmail y efectivamente, no vi que se estuvieran haciendo peticiones a estas URLs definidas aquí por lo que tal vez ni si quiera vale mucho la pena tener custom fonts.

¿Me fijo que se usa y solo hago la carga de eso?
¿Hago lo mismo para Poppins?

TextC0de · 2024-09-12T07:50:33Z

emails/templates/helpers/fonts.tsx

+  </>
+);
+
+const bebasNeue = [


Se agrego nueva fuente

TextC0de · 2024-09-12T07:53:22Z

src/datasources/email/sendTransactionalHTMLEmail.ts

@@ -40,14 +43,16 @@ const sendEmailFunction = async (
    logger.info("Using batch send");
    const resendArgsArray = resendArgs.map((args) => {
      const { from, htmlContent, subject, to, tags } = args;
-
-      return {
+      const next: CreateEmailOptions = {


Se tipó este mapeo para asegurar que todo está correcto con lo que espera resend

TextC0de · 2024-09-12T08:10:32Z

workers/transactional_email_service/index.tsx

-        {
-          name: purchaseOrder.user.name ?? purchaseOrder.user.username,
-          email: purchaseOrder.user.email,
+    if (communityInfo.name === "9punto5") {


Este if no me convence pero creo que es lo que hay por ahora

TextC0de · 2024-09-13T16:45:40Z

workers/transactional_email_service/index.tsx

+        subject: "Tus tickets están listos 🎉 | 9punto5",
+      });
+    } else {
+      await sendTransactionalHTMLEmail(this.resend, this.logger, {


Entiendo que este correo no está terminado aún ¿tal vez directamente sacar esto por ahora y sumarlo en otro PR?

Es necesario twekear el estilo, mejor dejarlo si no necesitamos borarlo

emails/templates/tickets/purchase-order-successful/9punto5.tsx

TextC0de · 2024-09-13T21:35:58Z

src/schema/purchaseOrder/actions.tsx

-  email,
-}: {
-  DB: ORM_TYPE;
+type SendPurchaseOrderSuccessfulEmailArgs = {


Creo que hay optimizar la aplicación para evitar hacer consultas repetidas.

Esta función era utilizada en createPaymentIntent si todos los tickets son gratis.

En createPaymentIntent se está utilizando la función "fetchPurchaseOrderInformation" que ya traía la información de los ticketTemplate, prices y currencies.

En esta función se estaba repitiendo la misma consulta.

Por eso en principio había hecho este cambio, para evitar consultas duplicadas a la BD.

TextC0de · 2024-09-14T04:22:06Z

emails/templates/helpers/fonts.tsx

@@ -237,27 +237,200 @@ export const PoppinsFont = () => (
  </>
 );

+const roboto = [


Seguí el estilo de lo mismo que se está haciendo con poppins, en la Poppins se está haciendo lo mismo de cargar un montón de fuentes.

Entiendo que sí una font-face no es utilizada no se descarga:

https://jaketrent.com/post/when-font-face-is-downloaded/

Igual estuve averiguando y vi que Gmail no soporta custom web fonts:

https://stackoverflow.com/a/70455614

https://www.caniemail.com/features/css-at-font-face/

https://react.email/docs/components/font

Lo recomendable es usar web fonts que están disponibles en la mayoría de clientes de correo.

Hice unos tests en Gmail y efectivamente, no vi que se estuvieran haciendo peticiones a estas URLs definidas aquí por lo que tal vez ni si quiera vale mucho la pena tener custom fonts.

src/schema/purchaseOrder/actions.tsx

emails/templates/tickets/purchase-order-successful/jschile.tsx

emails/templates/tickets/purchase-order-successful/9punto5.tsx

fforres · 2024-09-23T16:14:46Z

src/authz/helpers.ts

@@ -35,6 +35,9 @@ const isOwnerOfPurchaseOrder = async ({
  const isOwner = await DB.query.purchaseOrdersSchema.findFirst({
    where: (po, { eq, and }) =>
      and(eq(po.id, purchaseOrderId), eq(po.userId, user.id)),
+    columns: {


🤘🏼 me parece overkill, pero no estoy en contra

fforres · 2024-09-23T16:48:37Z

workers/transactional_email_service/index.tsx

+        subject: "Tus tickets están listos 🎉 | 9punto5",
+      });
+    } else {
+      await sendTransactionalHTMLEmail(this.resend, this.logger, {


Es necesario twekear el estilo, mejor dejarlo si no necesitamos borarlo

fforres · 2024-09-23T16:48:46Z

workers/transactional_email_service/index.tsx

-        {
-          name: purchaseOrder.user.name ?? purchaseOrder.user.username,
-          email: purchaseOrder.user.email,
+    if (communityInfo.name === "9punto5") {


fforres · 2024-09-23T16:56:13Z

src/schema/purchaseOrder/actions.tsx

-  email,
-}: {
-  DB: ORM_TYPE;
+type SendPurchaseOrderSuccessfulEmailArgs = {


Sacar la informacion de la BDD tiene q ser... 150ms? al ojo? Por ahora, no buscaría optimizar llamadas o performance (aka, evitar hacer consultas, etc) sino, hacer la experiencia de desarrollo mas facil por ahora.

Si esta, dale dejemoslo, pero 300ms no van a causar un drop de uso.

TextC0de added 4 commits September 12, 2024 04:25

feat: only fetch one column for isOwnerOfPurchaseOrder

89538d4

feat: add replyTo arg to sendEmailFunction

158898d

feat: add currency relation to purchaseOrders

ddf2eec

feat: send purchaseOrder success email on order status paid

1e62258

TextC0de commented Sep 12, 2024

View reviewed changes

emails/templates/helpers/fonts.tsx Show resolved Hide resolved

fix: transactionalEmailService not present on purchar_order_payment_s…

22bace6

…ync_cron

TextC0de changed the title ~~Agregar envío de correo de compra completada para 9punto5~~ checkPurchaseOrderStatus y correo de compra completada en 9punto5 Sep 12, 2024

TextC0de and others added 4 commits September 12, 2024 08:17

Merge branch 'main' into textcode/order-success-email

4b000bb

Signed-off-by: Ignacio Guzmán <[email protected]>

fix: prettier

d71ead9

feat: RPC_SERVICE_EMAIL in purchases orders cron

fb7730f

chore: remove null email service on syncPurchaseOrderPaymentStatus

7fd1471

fforres reviewed Sep 13, 2024

View reviewed changes

emails/templates/tickets/purchase-order-successful/9punto5.tsx Outdated Show resolved Hide resolved

fforres reviewed Sep 13, 2024

View reviewed changes

emails/templates/tickets/purchase-order-successful/jschile.tsx Outdated Show resolved Hide resolved

fforres reviewed Sep 13, 2024

View reviewed changes

src/schema/purchaseOrder/actions.tsx Outdated Show resolved Hide resolved

fforres reviewed Sep 13, 2024

View reviewed changes

joseglego reviewed Sep 13, 2024

View reviewed changes

TextC0de added 2 commits September 14, 2024 15:55

feat: use direct cloudflare image public url instead of assetURL

fc3dfb9

chore: rename PurchaseOrderSuccessfulJSChile to PurchaseOrderSuccessful

7a327ff

TextC0de commented Sep 17, 2024

View reviewed changes

fforres approved these changes Sep 23, 2024

View reviewed changes

Merge branch 'main' into textcode/order-success-email

c450524

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

checkPurchaseOrderStatus y correo de compra completada en 9punto5 #269

checkPurchaseOrderStatus y correo de compra completada en 9punto5 #269

TextC0de commented Sep 12, 2024

github-actions bot commented Sep 12, 2024 •

edited

Loading

fforres Sep 13, 2024

TextC0de Sep 13, 2024

fforres Sep 23, 2024

fforres Sep 13, 2024

TextC0de Sep 13, 2024

fforres Sep 23, 2024

fforres Sep 13, 2024

TextC0de Sep 13, 2024

TextC0de Sep 13, 2024

joseglego Sep 13, 2024

fforres Sep 14, 2024

TextC0de Sep 14, 2024

TextC0de Sep 17, 2024

TextC0de Sep 12, 2024

TextC0de Sep 12, 2024

TextC0de Sep 12, 2024

fforres Sep 23, 2024

TextC0de Sep 13, 2024

fforres Sep 23, 2024

TextC0de Sep 13, 2024

TextC0de Sep 14, 2024

fforres Sep 23, 2024

fforres Sep 23, 2024

fforres Sep 23, 2024

fforres Sep 23, 2024

		@@ -419,14 +413,6 @@ export const createPaymentIntent = async ({

		const userTicketsIds = userTickets.map((t) => t.id);

		await sendConfirmationEmail({

checkPurchaseOrderStatus y correo de compra completada en 9punto5 #269

Are you sure you want to change the base?

checkPurchaseOrderStatus y correo de compra completada en 9punto5 #269

Conversation

TextC0de commented Sep 12, 2024

github-actions bot commented Sep 12, 2024 • edited Loading

Coverage Report

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

github-actions bot commented Sep 12, 2024 •

edited

Loading