STS-Rosario / carpoolear_backend

The open source Laravel backend for the argentinian carpooling application: Carpoolear
https://carpoolear.com.ar
GNU General Public License v3.0
58 stars 45 forks source link

Add reCAPTCHA backend #135

Closed GonzaloGM closed 6 months ago

GonzaloGM commented 7 months ago

This will only allow new registrations only if they have reCAPTCHA token for now. Later we'll add capability to register via the app with email confirmation (until we finish PWA and can only allow reCAPTCHA in all).

p4bl1t0 commented 7 months ago

@GonzaloGM veo que así como lo tenemos ... sin captcha no podemos avanzar ... yo cambiaría que aún se pueda avanzar sin el "token" del catpcha ... el hecho de tenerlo lo que debería hacer es poner "active" en true y no generar el "activation_token". En cambio si el token no está (no hacemos la validación de reCaptcha) el usuario no estaría activo y solo podría activarse vía email.

GonzaloGM commented 7 months ago

@p4bl1t0 Buen punto! Ahí metí el cambio ese y armé la PR del frontend también: https://github.com/STS-Rosario/carpoolear/pull/95

p4bl1t0 commented 6 months ago

Perfecto