Skip to content

Elimina el atributo required del campo codpais en EditContacto#1909

Merged
NeoRazorX merged 1 commit intoNeoRazorX:masterfrom
daniel89fg:3382-contacto-pais-opcional
Mar 13, 2026
Merged

Elimina el atributo required del campo codpais en EditContacto#1909
NeoRazorX merged 1 commit intoNeoRazorX:masterfrom
daniel89fg:3382-contacto-pais-opcional

Conversation

@daniel89fg
Copy link
Copy Markdown
Contributor

Se permite que el campo codpais quede vacío, ajustándose a escenarios donde no se requiere un valor obligatorio. Esto previene errores en casos específicos como selecciones incompletas de país.

¿Cómo has probado los cambios?

Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado:

  • He revisado mi código antes de enviarlo.
  • He probado que funciona correctamente en mi PC.
  • He probado que funciona correctamente con una base de datos vacía.
  • He ejecutado los tests unitarios.

Se permite que el campo `codpais` quede vacío, ajustándose a escenarios donde no se requiere un valor obligatorio. Esto previene errores en casos específicos como selecciones incompletas de país.
Copilot AI review requested due to automatic review settings March 13, 2026 15:47
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Este PR permite que el campo codpais (país) en la vista de edición de contactos no sea obligatorio, para soportar escenarios donde el país puede quedar sin informar y evitar bloqueos por validación del formulario.

Changes:

  • Eliminado required="true" del widget select asociado a fieldname="codpais" en EditContacto.

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

@NeoRazorX NeoRazorX merged commit 28a4649 into NeoRazorX:master Mar 13, 2026
25 of 26 checks passed
@daniel89fg daniel89fg deleted the 3382-contacto-pais-opcional branch March 16, 2026 08:28
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.

3 participants