Traer detalles de una novedad - GET /api/news/{id}

raulbajales commented 12 years ago

Esto es para traer la "3er columna" de nuestro mockup:

lalala

Aca sugiero arrancar por lo mas facil, yo me imaginaba en principio traer solamente un poco mas de info sobre la publicacion en cuestion, el "preguntador" y la pregunta

El JSON podria ser algo como:

{
  "item": {
    "desc": "una bici loca",
    "pictureUrl": "http://lalala.com/sarasa.jpg"
  },
  "buyer": {
    "fullName": "Jose Lagarcha",
    "pictureUrl": "http://lalala.com/sarasa.jpg"
  },
  "question": "te quedan en color rojo?"
}

Cuando lo tengamos, tenemos que updatear la wiki.

gzanussi commented 12 years ago

Perfecto Raul, con esto ya puedo empezar . En cuanto soluciones la integración con maven :).

2012/9/26 Raul Bajales notifications@github.com

Esto es para traer la "3er columna" de nuestro mockup:

[image: lalala]https://a248.e.akamai.net/camo.github.com/49b98bfdcf0cd90e15a297c24e3446eb078eb68f/687474703a2f2f7075752e73682f3138534574

Aca sugiero arrancar por lo mas facil, yo me imaginaba en principio traer solamente un poco mas de info sobre la publicacion en cuestion, el "preguntador" y la pregunta

El JSON podria ser algo como:

{ "item": { "desc": "una bici loca", "pictureUrl": "http://lalala.com/sarasa.jpg" }, "buyer": { "fullName": "Jose Lagarcha", "pictureUrl": "http://lalala.com/sarasa.jpg" }, "question": "te quedan en color rojo?"}

— Reply to this email directly or view it on GitHubhttps://github.com/raulbajales/napkin/issues/48.

raulbajales commented 12 years ago

oki dale, eso lo tengo pronto, les aviso!

2012/9/26 German Zanussi notifications@github.com

Perfecto Raul, con esto ya puedo empezar . En cuanto soluciones la integración con maven :).

2012/9/26 Raul Bajales notifications@github.com

Esto es para traer la "3er columna" de nuestro mockup:

[image: lalala]< https://a248.e.akamai.net/camo.github.com/49b98bfdcf0cd90e15a297c24e3446eb078eb68f/687474703a2f2f7075752e73682f3138534574>

Aca sugiero arrancar por lo mas facil, yo me imaginaba en principio traer solamente un poco mas de info sobre la publicacion en cuestion, el "preguntador" y la pregunta

El JSON podria ser algo como:

{ "item": { "desc": "una bici loca", "pictureUrl": "http://lalala.com/sarasa.jpg" }, "buyer": { "fullName": "Jose Lagarcha", "pictureUrl": "http://lalala.com/sarasa.jpg" }, "question": "te quedan en color rojo?"}

— Reply to this email directly or view it on GitHub< https://github.com/raulbajales/napkin/issues/48>.

— Reply to this email directly or view it on GitHubhttps://github.com/raulbajales/napkin/issues/48#issuecomment-8901722.

Raul http://www.linkedin.com/in/raulbajales

gzanussi commented 12 years ago

@raulbajales @sulika Que tan necesario es tener el pictureUrl en el buyer? porque lo único que tiene asi parecido es el logo y la mayoria esta en null.

gzanussi commented 12 years ago

Otra pergunta, el path no debería ser /api/news/question/{id} ? porque cuando tengamos otros tipos de news, por ejemplo el de item, no existe el buyer.

raulbajales commented 12 years ago

ok al cambio del path, tiene sentido

lo del pictureUrl podes traerlo igual aunque sea null? si no es mucho laburo, sino dejalo

On Oct 17, 2012, at 1:52 PM, German Zanussi notifications@github.com wrote:

Otra pergunta, el path no debería ser /api/news/question/{id} ? porque cuando tengamos otros tipos de news, por ejemplo el de item, no existe el buyer.

— Reply to this email directly or view it on GitHub.

raulbajales commented 12 years ago

Lo del pictureUrl es solo para mostrar algo mas copado en la ui, aunque en la mayoria de los casos sea null (en cuyo caso podemos mostrar un default tipo esto http://puu.sh/1fOn3, creo que ayuda bastante a la ui

en cualqueir caso, si no es trivial para vos hacerlo, devolvelo con un null hardcodeado y mas adelante vemos de iterar esta parte

2012/10/17 raul.bajales@gmail raul.bajales@gmail.com

ok al cambio del path, tiene sentido

lo del pictureUrl podes traerlo igual aunque sea null? si no es mucho laburo, sino dejalo

On Oct 17, 2012, at 1:52 PM, German Zanussi notifications@github.com wrote:

Otra pergunta, el path no debería ser /api/news/question/{id} ? porque cuando tengamos otros tipos de news, por ejemplo el de item, no existe el buyer.

— Reply to this email directly or view it on GitHubhttps://github.com/raulbajales/napkin/issues/48#issuecomment-9534744.

Raul http://www.linkedin.com/in/raulbajales

sulika commented 12 years ago

pictureUrl: Note lo mismo. La mayoria esta en null. Creo que suma tenerlo por un tema de que si esta ahi y alguien lo esta usando, podria dar una sensacion de integracion con el sitio. Es la unica razon que le veo.
path: me gusta mas que el anterior pero difiero en algo... no me gusta que diga "news". Para mi tiene sentido un servicio que diga /api/news/ALGO solo si esta relacionado al reordenamiento, entrega de datos al usuario y novedades o eventos que estan ocurriendo. Para mi deberia ser un /api/dialog/... y todo lo que este relacionado con dialog deberia agrupar conversaciones de muchos tipos, en este caso, una pregunta. En otro caso (una venta) te mostraria un dialogo de venta (y asi por cada caso).

raulbajales commented 12 years ago

mmm @gzanussi ahora que lo pienso "/api/news/question/{id}" no me parece "completo", aunque si me parece correcto... (y lo cambiaria por "conversation" o "dialog" como sugiere tute)

teniendo en cuenta que los paths representan una jerarquia de recursos que estamos exponiendo via rest, me los imagino asi:

/api/news
- Expone el newsfeed (listado de novedades), un GET a esto nos da el JSON para dibujar nuestro newsfeed
/api/news/{id}
- Expone los detalles de una novedad, un GET a esto nos podria dar un JSON como en el definido en este mismo issue, y serviria para popular la "3er columna"

Tambien podriamos quere exponer:

/api/news/{id}/item
/api/news/{id}/buyer
/api/news/{id}/conversation

... y eso seria simplemente desdoblar el resource /api/news/{id} , la verdad es que esto no lo necesitamos (hoy) pero lo tiro para compartir como veo la jerarquia... @sulika por eso mepa que "/api/dialog/" no seria completamente correcto, aunque si "/api/news/{id}/dialog" (o "/api/news/{id}/conversation")

en cualqueir caso me parece mejor ir exponiendo solamente lo que vayamos necesitando (en el futuro podemos hacer todo "mas" REST y agregar soporte para otros verbos http para los recursos arriba descritos, pero no hoy)

@sulika @gzanussi solo queria exponer como veo la jerarquia que estamos por exponer via rest para ver que les parecia, modifiquenlo a piacere y en cualqueir caso dejemos claro finalmente cuales son los paths (en la wiki)

gzanussi commented 12 years ago

@raulbajales @sulika

Raul, cambiar los paths de nuestro lado son dos segundos, así que por mi lo cambiamos mas adelante si quieren, yo te aviso por las dudas si despues cambiarlo en el cliente se complica

raulbajales commented 12 years ago

Dale, en frontend cambiarlo es 1 segundo :D

asi que por mi no problem!

2012/10/17 German Zanussi notifications@github.com

@raulbajales https://github.com/raulbajales @sulikahttps://github.com/sulika

Raul, cambiar los paths de nuestro lado son dos segundos, así que por mi lo cambiamos mas adelante si quieren, yo te aviso por las dudas si despues cambiarlo en el cliente se complica

— Reply to this email directly or view it on GitHubhttps://github.com/raulbajales/napkin/issues/48#issuecomment-9537837.

Raul http://www.linkedin.com/in/raulbajales

gzanussi commented 12 years ago

@raulbajales @sulika Otra cosa, viendo el mockup, no faltaría la respuesta a la pregunta?

sulika commented 12 years ago

Si, falta.

Es para otro thread, habro un issue y lo seguimos por ahi.

On Wed, Oct 17, 2012 at 3:26 PM, German Zanussi notifications@github.comwrote:

@raulbajales https://github.com/raulbajales @sulikahttps://github.com/sulika Otra cosa, viendo el mockup, no faltaría la respuesta a la pregunta?

— Reply to this email directly or view it on GitHubhttps://github.com/raulbajales/napkin/issues/48#issuecomment-9538233.

Matias Sulik

sulika commented 12 years ago

El issue para hablar lo de respuestas a preguntas es el nro 58

On Wed, Oct 17, 2012 at 3:29 PM, Matias Sulik matias.sulik@gmail.comwrote:

Si, falta.

Es para otro thread, habro un issue y lo seguimos por ahi.

On Wed, Oct 17, 2012 at 3:26 PM, German Zanussi notifications@github.comwrote:

@raulbajales https://github.com/raulbajales @sulikahttps://github.com/sulika Otra cosa, viendo el mockup, no faltaría la respuesta a la pregunta?

— Reply to this email directly or view it on GitHubhttps://github.com/raulbajales/napkin/issues/48#issuecomment-9538233.

Matias Sulik

Matias Sulik

raulbajales commented 12 years ago

@gzanussi en el atributo "questions", podrias devolver un array en vez de un mapa?

hoy esta asi:

                    "questions": {
                        "2496684004": {
                            "text": "hola cuanto sale el envio hasta neuquen? saludos",
                            "answer": "Hola! hasta tu domiclio sale $75 por correo argentino"
                        }
                    }

podemos generarlo asi?

                    "questions": {
                        [{
                                            "id": "2496684004",
                            "text": "hola cuanto sale el envio hasta neuquen? saludos",
                            "answer": "Hola! hasta tu domiclio sale $75 por correo argentino"
                         },
                                         {
                                            "id": "2496684005",
                            "text": "otra preg",
                            "answer": null
                         }]
                    }

raulbajales commented 12 years ago

btw, es solo una cuestion de maejar las colecciones como array en cualqueir caso (para el endpoint api/news estamos devolviendo un array, por eso te pido si se puede hacer lo mismo en este caso)

raulbajales commented 12 years ago

ahhhh!!!! me equivoque!!!

@gzanussi me gustaria tenerlo asi mira, directamente el array (como en api/news):

                    "questions": [
                        {
                                            "id": "2496684004",
                            "text": "hola cuanto sale el envio hasta neuquen? saludos",
                            "answer": "Hola! hasta tu domiclio sale $75 por correo argentino"
                         },
                                         {
                                            "id": "2496684005",
                            "text": "otra preg",
                            "answer": null
                         }
                    ]

sulika commented 12 years ago

@raulbajales confirmame que asi es como lo necestias (lo revise y creo que esta bien).

Aviso que la query no esta tirando bien los resultados. @gzanussi esta trabajando en esto.

raulbajales / ventalandia-meli

Traer detalles de una novedad - GET /api/news/{id} #48