Nueva oferta laboral requiere telefono

[tzulberti]

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

[facundobatista]

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

[guidomperosino]

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.

[tzulberti]

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

[facundobatista]

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!!

[JigyasuRajput]

Hi! I’m looking into the issue, and I’d like to confirm the expected behavior...

When creating a job offer, should the form allow submission if at least one of the contact fields (phone, email, or URL) is filled? Also, would you like the error message to highlight a specific field that’s missing, rather than showing all fields as mandatory?

[facundobatista]

Hey! No field is mandatory per se, but you need at least one of them, yes.

So, if all are empty, an error should be raised. But no specific field is mandatory.