Skip to content

refactor(models): agregar campos de perfil al modelo User#5

Merged
Ochoa-Stack merged 1 commit into
developfrom
refactor/user-profile-fields
Jun 9, 2026
Merged

refactor(models): agregar campos de perfil al modelo User#5
Ochoa-Stack merged 1 commit into
developfrom
refactor/user-profile-fields

Conversation

@Ochoa-Stack

Copy link
Copy Markdown
Owner

refactor/user-profile-fields

Descripción

Se amplía la entidad User incorporando los campos first_name y last_name (String de 50 caracteres, estrictamente no nulos). Esta reestructuración es preventiva para soportar la futura interfaz del panel de configuración y la personalización en el envío de correos.

Detalles de la modificación:

  • Los campos se inyectaron lógicamente entre email y password_hash para mantener la cohesión visual de los datos de identidad frente a los datos de acceso.
  • Se actualizó el método __repr__ para que devuelva el formato <User {first_name} {last_name} ({email})>, facilitando la trazabilidad y el debugging interactivo.
  • La integridad del modelo y las restricciones de base de datos (CheckConstraint de roles) permanecen intactas.
  • Al no existir aún un estado físico de la base de datos ni historial de migraciones (alembic), esta modificación se asimila directamente en el esquema primario sin generar conflictos de esquema.

Tipo de cambio

  • feat
  • fix
  • refactor
  • chore
  • docs
  • test

Cómo probar

  1. Tras el merge, actualizar la rama base localmente: git checkout develop && git pull origin develop
  2. Navegar a la raíz del backend: cd backend
  3. Asegurar que el entorno virtual esté activo y abrir la interfaz interactiva de la aplicación:
    flask --app run shell
  4. Ejecutar la importación y verificar que la clase reconoce los nuevos atributos sin levantar excepciones:
    from app.models import User
    print(User.first_name)
    print(User.last_name)

Copilot AI review requested due to automatic review settings June 9, 2026 23:17
@Ochoa-Stack Ochoa-Stack changed the title refactor(models): add first_name and last_name to user model refactor(models): agregar campos de perfil al modelo User Jun 9, 2026
@Ochoa-Stack Ochoa-Stack merged commit 0e6072e into develop Jun 9, 2026
1 check passed

Copilot AI 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.

Pull request overview

This PR refactors the backend User ORM model to support upcoming user profile personalization by adding explicit first_name and last_name identity fields and improving debug readability via a custom __repr__.

Changes:

  • Add first_name and last_name columns to User as non-null String(50) fields.
  • Add a User.__repr__ that formats instances as <User {first_name} {last_name} ({email})>.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Ochoa-Stack Ochoa-Stack deleted the refactor/user-profile-fields branch June 9, 2026 23:20
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