search

podemos-info / participacion

GNU Affero General Public License v3.0
1 stars 2 forks source link

Social media tags still had some references to Madrid's participa site. #3

Closed andion closed 9 years ago

andion commented 9 years ago

Update Consul references like @abriendomadrid and "Madrid Participación" in _social_media_tag. When sharing something on twitter the source was @abriendomadrid. Using @plazapodemos.

TODO: fb:app_id should be updated too.

andion commented 9 years ago

One more TO-DO: It would be nice to use a custom social-media-icon.png this is the one being used right now: https://plaza.podemos.info/social-media-icon.png

You can change it under public/social-media-icon.png

PhoenixAlx commented 9 years ago

No sabía como contactar con vosotros. Así que lo digo por aquí.

Se han descuadrado medidas y comentarios.

Por ejemplo la que anteriormente era la 199 https://plaza.podemos.info/medidas/199

ahora es la https://plaza.podemos.info/medidas/201

Pero los comentarios siguen en la 199

joker-x commented 9 years ago

@PhoenixAlx los ids no coincidían con los de la votación en Ágora cuando lanzamos. Al arreglarlo teníamos dos opciones o borrar todos los comentarios hasta la fecha y empezar de nuevo o dejarlos aunque bailen de medida, optamos por la segunda.

@andion Thanks a lot!

PhoenixAlx commented 9 years ago

Y no se pueden arreglar ? me refiero los comentarios no tienen un id asociado al post o debate o tienen algún tipo de relación númerica en plan si le quitas 2 o 3 a su id no se ajustarían?

joker-x commented 9 years ago

Si, claro, pero lleva tiempo y hay bugs más urgentes.

PhoenixAlx commented 9 years ago

si me decis qué hay que mirar y que tocar, lo miro. A mi este me parece muy urgente porque en mucha medidas en los comentarios se estaban detallando y ahora la gente al leer esos comentarios se puede liar.

Y de ruby se poco o nada. Se python y sql por si os vale

andion commented 9 years ago

@PhoenixAlx Para mayor visibilidad o pedir Pull Requests puedes añadir una incidencia de forma separada en el repo: https://github.com/podemos-info/participacion/issues Con copiar la info que proporcionas (quizás añadir una explicación más detallada o capturas) llegaría.

De todas formas por lo poco que sé parece un caso... ¿aislado? Parece que alguien añadió una medida nueva, que por algún motivo sustituye a la antigua. Para ello el que ha editado la información tendría que haber editado la propuesta (si ha pasado el número máximo de votos para que se pueda editar desde administración o desde consola) en lugar de añadir una nueva.

Para este caso concreto bastaría con que algún administrador moviese (desde consola, por ejemplo) los comentarios del modelo medidas con id 199 al de id 201. Aquí no te puedo ayudar, pero bastaría con que el que corresponda ejecute:

 Comment.where(commentable_type: 'Medida', commentable_id: 199).update_all(commentable_id: 201)

De todas formas no es recomendable hacer este tipo de cambios en producción, trátese con cuidado.

PD: Es una tontería, pero el modelo Medidas no sigue la nomenclatura de AyuntamientoMadrid/participacion en la que los modelos, controladores y rutas se ponen en inglés. Podríamos añadirlo como incidencia de bajo nivel si alguien lee hasta aquí y está de acuerdo.

PhoenixAlx commented 9 years ago

Muchas gracias por la respuesta @Andion pasa en más...supongo que al moverse alguna medida ha arrastrado al resto.

De acuerdo con lo que propones :)

joker-x commented 9 years ago

Si, hay más ids cruzados y hay que contar con que los comentarios realizados después del cambio están bien y no hace falta moverlos.

@PhoenixAlx @andion trasladamos el debate a https://github.com/podemos-info/participacion/issues/11 si os parece

joker-x commented 9 years ago

@andion abre todos los issues que estimes convenientes y muchas gracias :D

PhoenixAlx commented 9 years ago

Genial :)