pyarweb icon indicating copy to clipboard operation
pyarweb copied to clipboard

Nueva oferta laboral requiere telefono

Open tzulberti opened this issue 2 years ago • 4 comments

Cuando se crea una nueva oferta laboral, el telefono figura como obligatorio. Revisar porque pasa esto y si realmente es necesario

tzulberti avatar Jan 14 '23 14:01 tzulberti

Hola! El teléfono no debería ser obligatorio, pero sí se tiene que incluir alguna forma de contacto...

A nivel modelo, tenemos que...

    contact_phone = models.CharField(
        max_length=255, null=True, blank=True, verbose_name=_('Teléfono')
    )

(null y blank en True)

Y luego tenemos estre constraint:

            models.CheckConstraint(
                name='%(app_label)s_%(class)s_not_all_contact_info_null',
                check=(
                    models.Q(
                        contact_mail__isnull=False,
                    )
                    | models.Q(
                        contact_phone__isnull=False,
                    )
                    | models.Q(
                        contact_url__isnull=False,
                    )
                ),
            ),

Más allá de eso, habría que ver como se presenta la info en el form...

facundobatista avatar Jan 28 '23 23:01 facundobatista

Hola, en el form de creación de Job Offer tenemos esta validación (método clean)

        if not any([contact_mail, contact_phone, contact_url]):
            # Highlight all involved the fields
            self.add_error('contact_mail', '')
            self.add_error('contact_phone', '')
            self.add_error('contact_url', '')

            raise ValidationError(_('Debe ingresar al menos un dato de contacto.'))

Entiendo que solo sería necesario uno de los campos.

guidomperosino avatar Feb 12 '23 22:02 guidomperosino

Entiendo que solo sería necesario uno de los campos.

Si es como vos decis. El tema es que es medio confuso el error si no se ve el error arriba del todo en el form

tzulberti avatar Feb 12 '23 23:02 tzulberti

Ah, entiendo el punto de confusión... el form "da error", el mensaje queda arriba de todo y a uno se le escapa, pero los tres campos figuran "en error" y parece como si fueran todos obligatorios...

Deberíamos mejorar el error mostrado en los campos en sí. Confirmame @tzulberti que es así como digo, y preparo un branchito... gracias!!

facundobatista avatar Feb 14 '23 22:02 facundobatista