PyAr / pyarweb

El sitio web de Python Argentina
http://python.org.ar
Apache License 2.0
81 stars 99 forks source link

Nueva oferta laboral requiere telefono #556

Open tzulberti opened 1 year ago

tzulberti commented 1 year ago

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

facundobatista commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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