Responsiveness #100
Responsiveness #100
Conversation
There was a problem hiding this comment.
@maurycortes Buen ttrabajo en encontrar y corregir los errores ortograficos 👍
Aunque no se puede hacer merge de esta branch a master porque tiene conflictos, creo que te falto hacer pull origin master en tu local y de ahi hacer una branch, esta branch esta 15 commits atras de master y esta causando conflictos, igual lo checamos en clase. Lo demas buen trabajo, igual despues de arreglar los errores de merge y poner al corriente la branch ya checo completamente el resultado bajando a local y probandolo, de mientras hay que esperar a arreglar las cosas de conflicto 👍
| @@ -28351,3 +28351,141 @@ Completed 200 OK in 104ms (Views: 100.2ms | ActiveRecord: 1.0ms) | |||
| >>>>>>> defaz-datos-iniciales | |||
|
|
|||
|
|
|||
| Started GET "/pacientes_pages/mis_sesiones" for 127.0.0.1 at 2019-03-01 17:45:26 -0600 | |||
There was a problem hiding this comment.
Estos conflictos ya no deberian de aparecer ya que esta en .gitignore este archivo, tienes una branch muy vieja, debes de actualizar primero tu branch antes de hacer cambios men.
| @@ -286,4 +286,4 @@ DEPENDENCIES | |||
| web-console (>= 3.3.0) | |||
|
|
|||
| BUNDLED WITH | |||
| 1.16.1 | |||
| 2.0.1 | |||
There was a problem hiding this comment.
@maurycortes No soy experto en rails pero no puedes mover versiones asi nadamas o porque decidiste cambiarlas a esas versiones? Cambiar versiones asi repentinamente puede traer errores a la aplicacion si no se checan antes, se que apenas estas aprendiendo rails pero no creo que cambiar la version sea necesario aqui, ya que no fueron cambios mayores solo fueron de responsive, esto tampoco deberia de ir. 👍
| @@ -188,7 +188,7 @@ GEM | |||
| method_source | |||
| rake (>= 0.8.7) | |||
| thor (>= 0.18.1, < 2.0) | |||
| rake (12.3.1) | |||
| rake (12.3.2) | |||
There was a problem hiding this comment.
Estos cambios los hiciste tu o se hicieron solos? o te lo pidio tu enviorment? o hiciste update? debemos de checarlos en grupo primero para saber que implican las versiones que estamos usando o a las que nos vamos a mover.
| @@ -4,7 +4,7 @@ | |||
|
|
|||
| <div class ="row"> | |||
| <div class="col s12"> | |||
| <h4>Bienvenido al programa...</h4> | |||
| <h4>Bienvenido al programa</h4> | |||
There was a problem hiding this comment.
Este cambio ya esta en master hasta donde yo se, yo lo movi porque esta en este commit? creo que esto esta causando conflictos con el merge. Si hiciste pull master origin y despues hiciste tu branch de ese master men?
| @@ -1,5 +1,5 @@ | |||
| <div class="card" style="width: 90%; margin: auto; text-align="center"> | |||
| <h1>Sesiones paciente <%=getPacienteName(@paciente) %></h1> | |||
| <h4>Sesiones paciente <%=getPacienteName(@paciente) %></h4> | |||
There was a problem hiding this comment.
Creo que este cambio tambien ya esta en master
| <th>Cintura</th> | ||
| <th>IMC</th> | ||
| <th>G.V.</th> | ||
| <th>E.M.</th> | ||
| <th>Hora</th> | ||
| <th>Nutriologo</th> | ||
| <th>Nutriólogo</th> |
There was a problem hiding this comment.
Super bien que agregaste y corregiste los spellings que faltaban! 👍
| <th>Peso</th> | ||
| <th>Estatura</th> | ||
| <th>Grasa</th> | ||
| <th>Musculo</th> | ||
| <th>Músculo</th> |
There was a problem hiding this comment.
Este ya esta en la branch de master
| <thead> | ||
| <tr> | ||
| <th>Numero Sesion</th> | ||
| <th >Número Sesión</th> |
Changes in responsiveness
Size of titles
Spelling changes