search

mxabierto / madison

(sin mantenimiento) Madison 2.0 - nos fuimos a mxabierto/participa
GNU General Public License v3.0
2 stars 1 forks source link

Fix/translated controller messages #220

Closed codemoe closed 9 years ago

defvol commented 9 years ago

Depende de #219

defvol commented 9 years ago

como acordamos en la llamada, este PR necesita entradas de diccionario en lugar de strings hardcodeadas

codemoe commented 9 years ago

Listos los cambios @rodowi , Por favor revísalos y si ya quedó, para que puedas hacer el merge....

defvol commented 9 years ago

on it...

defvol commented 9 years ago

El 748241e debería estar en otro PR no?

codemoe commented 9 years ago

No.... Lo que pasa es que estaba imprimiendo en el controller los mensajes para ver que se cargaran correctamente del diccionario y en un commit se me fue esa línea así que en otro commit lo eliminé...

defvol commented 9 years ago

Ok, haré squash

defvol commented 9 years ago

Hay varias cosas por corregir. Dejé las anotaciones en el code review.

codemoe commented 9 years ago

@rodowi Listas las correcciones... Échale un vistazo...

defvol commented 9 years ago

Gracias @codemoe aprovecho para hacer squash de 5d1244c y be14651 y hago merge

defvol commented 9 years ago

Limpié el historial con un rebase:

hice squash de 2 commits duplicados y el de remover el debug message también hice squash del fix de identación con su commit anterior

defvol commented 9 years ago

Siempre es bueno agregar un screenshot como PoC:

screen shot 2015-05-07 at 12 25 27 pm

cc: @urkonn @codemoe ¿cómo ven?