#6427 - SMS Recover Password #2037

Open
jsegarra wants to merge 72 commits from 6427_sms_resetPassword into dev
2 changed files with 5 additions and 9 deletions
Showing only changes of commit 01f97188f4 - Show all commits

View File

@ -12,8 +12,7 @@
ng-model="$ctrl.verificationCode"
vn-name="verificationCode"
Outdated
Review

No entiendo el poner 2 vn-textfields con ifs.
Con poner `label="User, phone, or recovery email" sobraria.

Y tampoco se si deberia poder poner numeros de telefono para decir que son ellos (lo consultaria con Juan)

No entiendo el poner 2 vn-textfields con ifs. Con poner `label="User, phone, or recovery email" sobraria. Y tampoco se si deberia poder poner numeros de telefono para decir que son ellos (lo consultaria con Juan)

El teléfono se usa para validar la acción de recuperar la contraseña. Porque puede darse el caso que el usuario ponga su id y no le esté llegando el SMS porque en algún momento se equivocó de teléfono.

El teléfono se usa para validar la acción de recuperar la contraseña. Porque puede darse el caso que el usuario ponga su id y no le esté llegando el SMS porque en algún momento se equivocó de teléfono.
Outdated
Review

Entonces podría poner tu id, y mi numero de teléfono y te podría cambiar la contraseña?
Lo que se hacia con el correo es apartir del correo sacar el id del usuario. Supongo que con el telefono sera igual

Entonces podría poner tu id, y mi numero de teléfono y te podría cambiar la contraseña? Lo que se hacia con el correo es apartir del correo sacar el id del usuario. Supongo que con el telefono sera igual

No podrías, porque tu pones el id de usuario y teléfono, y si ambos valores no existen, no te envía SMS. En local puedes probar con el userId:9 que tiene el teléfono "432978106"

El teléfono de recuperación solo lo puede cambiar quien es propietario del registro, ya que tiene una validación del id del registro contra el id del usuario logeado

Pero vamos, que yo podría estar contaminado con el desarrollo, y a lo mejor tu consigues bordear la restricción. si es así, repórtamelo, por favor.

No podrías, porque tu pones el id de usuario y teléfono, y si ambos valores no existen, no te envía SMS. En local puedes probar con el userId:9 que tiene el teléfono "432978106" El teléfono de recuperación solo lo puede cambiar quien es propietario del registro, ya que tiene una validación del id del registro contra el id del usuario logeado Pero vamos, que yo podría estar contaminado con el desarrollo, y a lo mejor tu consigues bordear la restricción. si es así, repórtamelo, por favor.
autocomplete="false"
class="vn-mt-md"
>
class="vn-mt-md">
</vn-textfield>
<vn-one>
<vn-vertical class="vn-mb-sm">
@ -21,15 +20,13 @@
disabled="$ctrl.code"
jsegarra marked this conversation as resolved Outdated
Outdated
Review

Y aqui igual en vez de poner dos, pondria algo como. te enviaremos un mensaje por el tipo de envio elegido o algo asi

Y aqui igual en vez de poner dos, pondria algo como. te enviaremos un mensaje por el tipo de envio elegido o algo asi
label="Teléfono móvil"
val="sms"
ng-model="$ctrl.method"
>
ng-model="$ctrl.method">
</vn-radio>
<vn-radio
disabled="$ctrl.code"
label="E-mail"
val="email"
ng-model="$ctrl.method"
>
ng-model="$ctrl.method">
</vn-radio>
</vn-vertical>
</vn-one>

View File

@ -31,9 +31,8 @@
vn-one
label="Recovery phone"
ng-model="$ctrl.user.recoveryPhone"
disabled="$root.user.id !== $ctrl.user.id"
>
</vn-textfield>
disabled="$root.user.id !== $ctrl.user.id">
</vn-textfield>
jsegarra marked this conversation as resolved Outdated
Outdated
Review

Corregir tabulación

Corregir tabulación
<vn-autocomplete
label="Language"
ng-model="$ctrl.user.lang"