search

fos-alex / estamos-para-ayudarlo

1 stars 2 forks source link

Listas de Compras - Compartir #10

Closed pvillamil25 closed 10 years ago

pvillamil25 commented 10 years ago

Juli desde el lado mobile, Pablo desde el servicio Web

pmussari commented 10 years ago

Se creo un nuevo servicio POST http://HOSTWEB/app/compartir/IDLISTA IDLISTA : el id de la lista en el post va un JSON { "email" : "asdasd@asdasd.cc"}

Este servicio controla que exista la lista, que exista el usuario, y si esta todo bien lo agrega a la lista.

Faltaria la posibilidad de tener de alguna manera el usuario registrado en la APP MOBILE sin pasarlo por parametro (quiza por cookie), para que podamos hacer los filtrados correspondientes

jbichman commented 10 years ago

Pablito, estuve armando el Compartir lista... y noté dos cosas: 1) Probé compartir a varias direcciones de correo (mails registrados, mails no registrados, input vacío), y en todos los casos, el response me retorna código 403 y un mensaje relacionado a la falta de email. Corroboré en el envío del request desde el lado mobile, y aparentemente funciona bien. Adjunto imagenes de los casos. Habría que verificar qué pasa cuando le llega un mail, porque hay casos en que el usuario está registrado y debería compartirla la lista. O capaz me faltó hacer algo desde el lado mobile.

Response: issue10_response

Req. Post: issue10_2

2) El mensaje de error debería ser un poco mas específico: Error por usuario no registrado, por mail inválido por mail vacío, etc.... igual entiendo que en los casos que probé siempre salió por el mismo error. Habría que revisarlo

Casos testeados: issue10_1

pmussari commented 10 years ago

Juli,

Fijate que el json deberia ser {email:xxxxxx} , vos creo que estas mandando {mail:xxxxxx}.

Los errores estan especificados, cuando mandes un email que no tiene usuario registrado te lo informa.

Proba con ese cambio y despues cambiamos lo mensajes de error si te parece.

jbichman commented 10 years ago

Listo! ahi funciona joya Me lo vuelvo a asignar asi veo temas de diseño. Los mensajes están bien discriminados, los dejamos así. Gracias!

jbichman commented 10 years ago

TO-DO:

pvillamil25 commented 10 years ago

che, los mensajes de error juli que mostras lo hacen desde el servicio web no?

jbichman commented 10 years ago

sí, lo mandan desde el servicio web. igual la idea es centralizar todos los mensajes en un php o html de mensajes. Pero mientras funcione que quede así. Hay que modificar el archivo de trazabilidad con los mensajes que se vayan agregando.

jbichman commented 10 years ago

Detalles de diseño listo!