LeandroBraier / Klaukol

Aquí van todos los avances web y apps para nuestro potencial futuro cliente Klaukol
0 stars 0 forks source link

Bug en Carga de Imágenes en Novedades #359

Closed LeandroBraier closed 7 years ago

LeandroBraier commented 7 years ago

@mkreyness estamos teniendo problemas al cargar imágenes en novedades tanto en Parex como Revear (mismo CMS)

Revisás porfa qué pasa?

Caso de ejemplo: En la nota "Ahorrá en la factura: reducí el gasto energético con un diseño adecuado" en Revear cargué varias veces los campos de "Imagen" e "Imagen Grande" y nada. Da error.

Te fijás porfa? Porque nos está pasando en novedades parexklaukol también.

mkreyness commented 7 years ago

acabo de subir yo dos fotos cualquiera en esos campos sin ningun problema (en la noticia de "ahorra en la factura".

LeandroBraier commented 7 years ago

Puede ser que tenga tildes la imagen? me acabo de dar cuenta de eso.

Leandro Braier Producción Audiovisual & Social Media Benteveo Agencia Productora http://benteveo.tv Móvil 1158717399

On Tue, Sep 6, 2016 at 11:18 PM, mkreyness notifications@github.com wrote:

acabo de subir yo dos fotos cualquiera en esos campos sin ningun problema (en la noticia de "ahorra en la factura".

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/LeandroBraier/Klaukol/issues/359#issuecomment-245155230, or mute the thread https://github.com/notifications/unsubscribe-auth/AGaJgNEBLuFXqxYJBCneu4Ki147iKfdlks5qnh7dgaJpZM4J2dgJ .

mkreyness commented 7 years ago

ni idea porque ya los pise, pero seguro que se rompe tratar de uploadear un archivo con un nombre non-safe.

LeandroBraier commented 7 years ago

Perdón Marce. Ya lo pude resolver a eso. Ahora me encuentro con otro bug, este parece ser real.

Publico y la nota y aparece en el listado de Tendencias, pero el link falla:

http://www.revear.com.ar/tendencias/particular/ahorra-en-la-factura--reduci-el-gasto-energetico-con-un-diseno-adecuado.html

Te fijás qué puede ser?

Aprovecho a consultar. La tendencia que aparece en la Home se actualiza sola o hay que pedirle a Fede especialmente?

mkreyness commented 7 years ago

A que te referis con que el link falla ? Hice click ahi y me lo mostro perfecto. Nada es inmediato, el proceso que regenera todo corre cada 5 minutos. Dentro del momento que se regenera puede que este primero la lista de Tendencias y despues la nota en si, pero se ajusta en segundos.

mkreyness commented 7 years ago

lo que veo que queda horrible es que ya hay demasiadas "etiquetas", y sale todo superpuesto en esa parte.

mkreyness commented 7 years ago

justamente donde no aparece es al hacer click en la etiqueta, debe tener que ver con los acentos. Me fijo.

mkreyness commented 7 years ago

Lean, las etiquetas van separadas por PUNTOYCOMA, no por COMA. Por eso andan mal.

mkreyness commented 7 years ago

Fijate las primeras novedades cargadas, las tienen (correctamente) cargadas con la separacion con puntoycoma.

LeandroBraier commented 7 years ago

Me refiero a que, si tarda en generar el link, debería tardar lo mismo en mostrar como publicada la nota. Si entra un usuario justo en el mientras tanto queda en la nada. No se puede resolver eso?

Lo de las etiquetas ya lo corregí, es un chino acordarse de eso. Si se pudiera mejorar para que lo tome de ambas formas mejor. Sino hacer un campo por etiqueta?

mkreyness commented 7 years ago

Deberia ser todo suficientemente instantaneo, eso depende de la performance del server y la cantidad de cosas a procesar. Puedo ver si cambiando el orden en que se procesa esto se evita, pero hay cosas que se apuntan mutuamente, asi que no se si es realmente factible.

Se usó puntoycoma porque asi me lo habia pasado Fede cuando armaron los datos en base al sitio viejo de Klaukol, yo respete esa convención, creo. La ventaja del puntoycoma es que podes armar etiquetas que incluyan comas, por ejemplo: "Hugo, Paco y Luis; El Pato Donald; Mickey y Pluto" Si usaras coma, "Hugo" se separaira de "Paco y Luis'.

LeandroBraier commented 7 years ago

Ok, entiendo Mk. Dejemoslo así por ahora entonces. abrazo