Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rails3.2 #21

Open
wants to merge 34 commits into
base: master
Choose a base branch
from
Open

Rails3.2 #21

wants to merge 34 commits into from

Conversation

olvap
Copy link
Member

@olvap olvap commented Feb 29, 2012

falta ver lo de la migraciones

Olvap and others added 24 commits February 27, 2012 15:54
si no las sacas da error cuando queres correr las migraciones desde la version cero
… muevo las imagenes para que sea compatible con 3.2
Jquery se maneja por una gema y no es necesario tener los archivos en assets
Quedan los de Reserva porque están en uso.
De esta forma las migraciones quedan en el sistema.
El método que se usaba en las migraciones y daba error fue eliminado.
Cambio otras migraciones que afectaban las tablas de versionado.
Vuelvo también otra migración perdida de antes sobre movimientos.
@eloyesp
Copy link
Member

eloyesp commented Mar 2, 2012

Errores que encuentro con la app:

  • me da un error con {public → app/assets}/images/dropselectionleft.png y {public → app/assets}/images/dropselectionright.png (no lo encuentra)
  • no precompila los assets por un problema con el Gemfile (falta el grupo de assets?)
  • la "x" para cerrar las pestañas de reserva queda en la segunda linea.
  • Los formularios de New se salen del recuadro.
  • La carga de deposito no funciona "Entidad no puede estar en blanco"

@eloyesp
Copy link
Member

eloyesp commented Mar 2, 2012

Por otro lado, el tema de las migraciones quedó en la siguiente cuestión, habría que elegir entre arreglar las migraciones (para que creen realmente el schema de representaciones) o habría que empezar a usar el schema de representaciones (o sea no volver a correr las migraciones, sinó hacer rake db:setup (como recomienda rails)).

@eloyesp
Copy link
Member

eloyesp commented Mar 2, 2012

error de la x

@olvap
Copy link
Member Author

olvap commented Mar 4, 2012

la direccion de las imagenes tienen que ser app/assets/drop...png no app/assets/images/drop..png fijate que puede que anade por ahí el problema

@eloyesp
Copy link
Member

eloyesp commented Mar 5, 2012

Arreglado

  • arreglado el problema de la carga de la imágenes (que igual no cambia nada, pero da un error menos).
  • arreglado el encoding de los assets (ya que si usamos el asset pipeline da error)

Quedan:

  • la x que se ve mal. (no encontré que puede ser (los css son iguales en firebug), me imagino que es algún problema con los js)
  • el asset pipeline está medio descuajeringado y carga cosas 2 veces (eso hace que algunos js no anden)
    • Entre estos problemas está el de la carga de los datos de la reserva en el formulario de pagos.
  • Hay que actualizar el Gemfile para que tenga assets y ande el precompile.
  • Hay un test que da error en stringify keys (pero parece un error del test no de la app).

@eloyesp
Copy link
Member

eloyesp commented Mar 5, 2012

Cuando intento arreglar los assets las tablas no me funcan, así que me parece que el jqGrid es incompatible con la última versión de jQuery... da un error en jQuery.jgrid.stripHtml (is not a function, aunque la define el jqgrid). Parece ser un error pequeño.

Habilito el asset pipeline.
El archivo de traducción en jqgrid tiene que ser llamado antes que los otros
porque sinó no carga la tabla.
Quito las fuentes de app/assets.
@eloyesp
Copy link
Member

eloyesp commented Mar 5, 2012

Ya andan las tablas, el archivo de las traducciones de jqGrid está mal hecho, así que hay que cargarlo antes (o borra jqGrid :( ). ahora anda todo a través del asset pipeline. (no lo probé en production).

El problema de la x sigue ahí y la carga de datos no la probé aún...

@ghost ghost assigned eloyesp Mar 5, 2012
los params deben ser un hash no un ActiveRecord. :(
@eloyesp
Copy link
Member

eloyesp commented Mar 5, 2012

@olvap Podrás mirar a ver si encontras la causa del problema de las x (no encuentro nada de por qué puede ser eso...)

@ghost ghost assigned olvap Mar 8, 2012
@eloyesp
Copy link
Member

eloyesp commented Mar 8, 2012

El problema de entidad en Deposito (que no se carga) viene por el script en app/views/depositos/new.js.erb el cual por alguna causa no puedo debugear con firebug (por alguna causa no lo ve)... debe tener que ver con la actualización de jquery y de jquery-ujs.

Edit... !! se debe al upgrade de rails, que parece le cambia los id o class al formulario.

$("#deposito_submit").click(function(){
  $("#deposito_entidad_id").val($("#search_agency_id_or_operadora_id_eq").val());
});

Parece que #deposito_submit no tiene más esa clase. Si le pongo input[value='crear deposito'] anda...

@eloyesp
Copy link
Member

eloyesp commented Mar 9, 2012

El formulario ahora "anda" pero carga mal... y no se vuelve a cero (queda cargando después de submitear).

Prové actualizar formtastic, pero pasan dos cosas:

  1. Ese formulario no usa formtastic (a pesar de que dice semantic_form_for)
  2. Encontré muuchos problemas, así que me bajé del caballo, eso, queda para despues de rails 3.2.
  3. Me encontré con que movimiento sigue teniendo algo que ver con Cuenta, así que termine de matar a las cuentas.

@eloyesp
Copy link
Member

eloyesp commented Mar 9, 2012

Active admin usa kaminari en vez de will_paginate habría que ver de hacer el cambiaso.

https://github.com/BlueLemon/representaciones/blob/rails3.2/Gemfile#L14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants