Skip to content

feat: adición autoservicio para participantes - #5

Merged
jevillanueva merged 7 commits into
devfrom
feat/4-autoservicio
Oct 18, 2026
Merged

jevillanueva merged 7 commits into
devfrom
feat/4-autoservicio

Conversation

@jevillanueva

Copy link
Copy Markdown
Member

name: Pull Request Self Service
about: Pull request para el auto servicio
title: "[PR] Self Service Py Day"
labels: enhancement
assignees: "aasg2003"

✨ Descripción

Se adiciona la funcionalidad de auto servicio para el PyDay con la posibilidad de tener
autoservicio con los QR especialmente para eventos virtuales, y tambien ganar puntos y canjear premios digitales
de manera virtual

🔄 Tipo de cambio

  • 🐛 Bug fix - corrección de un problema existente
  • 🚀 Nueva funcionalidad - agrega algo nuevo sin romper lo existente
  • 💥 Breaking change - modifica comportamiento existente
  • 📄 Documentación - cambios solo en docs, README o comentarios
  • 🎨 Estilo / Refactor - mejoras de código sin cambiar funcionalidad
  • 🧪 Tests - agrega o corrige pruebas
  • 🔧 Infraestructura - cambios en CI/CD, Docker o configuración

🔗 Issue relacionado

#4

Closes #

🧩 ¿Qué cambiaste?

  • Agrega el flujo de autoservicio para participantes: registro con CI y fecha de nacimiento, generación de código y QR, consulta de puntos, validación de enlaces de checkpoints y registro de asistencia con puntos.
  • Incorpora consulta y canje de recompensas virtuales y regalos. El canje valida al participante, el saldo, la disponibilidad y el stock; entrega códigos de descuento o enlaces de descarga y envía un correo cuando corresponde.
  • Refactoriza la lógica compartida de canjes y actualiza las rutas de recompensas. Añade variables de configuración SMTP de ejemplo para producción, y configura un devcontainer y reglas de pre-commit.

🧪 ¿Cómo probarlo?

Probar mediante docs de fast api la seccion de self service

# Configura .env con la conexión a MongoDB y los valores necesarios.
# Levanta la API
docker compose -f docker-compose.dev.yaml up --build -d

# Consulta la documentación interactiva
# Abre http://localhost:8000/api/v1/docs

✅ Checklist

  • Mi código sigue los estándares del proyecto (PEP 8)
  • Probé los cambios localmente y funcionan
  • Agregué o actualicé tests para cubrir los cambios (si aplica)
  • Actualicé la documentación relevante (README, docstrings, comentarios)
  • No hay conflictos con la rama main
  • El título del PR sigue el formato: tipo: descripción breve (ej. feat: agregar validación de serie)
  • Vinculé el issue relacionado con Closes #N

💬 Notas para el revisor

Verificar especialmente la disponibilidad y el stock al canjear, la validación CI/código de participante y la configuración SMTP real del entorno.

feat: update Dockerfile to use Python 3.13, enhance VSCode settings, and add MongoDB extension

feat: implement self-service registration and QR code generation for participants

feat: implement self-service registration and QR code generation for participants

feat: add presentation checkpoint link management and self-service participation features
@jevillanueva
jevillanueva requested a review from AASG2003 October 6, 2026 00:17
@jevillanueva
jevillanueva changed the base branch from master to dev October 12, 2026 01:12
@AASG2003

Copy link
Copy Markdown

al mandar post api/v1/self-service/check/{code} para mandar puntos de una charla a una persona si hay exactamente 1 persona sin acreditacion, teniendo su control_code: "", reconoce y le asigna los puntos, igual cuando pregunta sus puntos en POST /api/v1/self-service/points

@AASG2003

Copy link
Copy Markdown

dentro de app/utils/control_code_generator.py esta documentado que el codigo al final usa un digito (adjetivo-nombre-digito), pero en la ejecucion solo usa (adjetivo-nombre), no tiene afectacion real pero es contradictorio.

@jevillanueva

Copy link
Copy Markdown
Member Author

@AASG2003 Corregido para nueva revisión

@AASG2003 AASG2003 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Todo bien

@jevillanueva
jevillanueva merged commit 91e0d34 into dev Oct 18, 2026
3 checks passed
@jevillanueva
jevillanueva deleted the feat/4-autoservicio branch October 18, 2026 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants