Om 554

[SebaKisser]

Summary by CodeRabbit

Notas de Lanzamiento

• Actualizaciones

  • Se implementó un nuevo sistema para el envío de correos de recuperación de contraseña.
  • Se agregaron límites de validación para los campos de precios en la creación y actualización de tiendas, estableciendo un máximo de 99.999.999,99 por campo.

• Chores

  • Se actualizaron las dependencias del proyecto para mejorar la gestión de correos electrónicos.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

El PR migra el envío de correos electrónicos de recuperación de contraseña desde Resend hacia SMTP utilizando nodemailer, agrega la dependencia correspondiente en package.json y actualiza las validaciones de precios en los DTOs de tienda para incluir límites máximos en los campos base_price y distance_price.

Changes

Migración a Email SMTP y Validación de Precios

Layer / File(s)	Summary
Dependencia nodemailer en package.json package.json	Se agrega nodemailer@^8.0.10 a las dependencias del proyecto.
Implementación SMTP en email.service.js src/lib/email.service.js	sendPasswordResetEmail configura un transporter nodemailer con variables de entorno EMAIL_* y elimina la validación explícita de errores de Resend, manteniendo la firma y estructura de respuesta equivalente.
Validación de precios en DTOs de tienda src/modules/global/dtos/commerce/store.request.dto.js	CreateStoreDTO y UpdateStoreDTO incorporan límite máximo de 99999999.99 en base_price y distance_price junto con validación nonnegative existente.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutos

Possibly related PRs

• CrisNAC/BackendMarketplace#169: Modifica la configuración del transporter nodemailer en src/lib/email.service.js agregando opciones de conectividad IPv4.
• CrisNAC/BackendMarketplace#194: Altera la lógica de sendPasswordResetEmail en src/lib/email.service.js modificando el proveedor de envío de emails.
• CrisNAC/BackendMarketplace#170: Ajusta las opciones del transporter nodemailer en src/lib/email.service.js para optimizar la conectividad.

Suggested reviewers

• leoAchu16
• Andoumeda

Poem

🐰 Whiskers de nodemailer danzando,
SMTP por los campos trotando,
Precios validados, límites llegando,
Resend adiós, nuevo camino andando,
Backend robusto, cambios confirmando! 📧✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	El título 'Om 554' es genérico y no describe el cambio principal del pull request; refiere solo a un identificador de rama sin aclarar qué se modifica.	Usar un título descriptivo que resuma el cambio principal, por ejemplo: 'Migrar envío de emails de Resend a SMTP con nodemailer' o 'Agregar validación de límites en precios de tiendas'.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch OM-554

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud