Video 2 #95
Video 2 #95
Conversation
There was a problem hiding this comment.
Good job 👍 2 comentarios.
-
Hay que limpiar el historial de commits y procurar no dejar todo lo que se cambio, sino dejar el resultado final.
-
vi que hiciste una branch (esta) derivada de la branch de hot fixes, Esta bien no pasa nada, solo que cuando se apruebe el PR si alguien mas cambio algo puede que tu branch tenga conflictos... y se van a tener que resolver, si quieres evitar esto hay de dos.
a. Cuando acabes tu branch (la de hot fixes por ejemplo) y quieres seguir en otra te recomendaria que pasar a master y derivaras la branch desde ahi si es que te acuerdas de los cambios que hiciste y todavia no tienes y ya despues haces unrebasede master a tu branch.
b. Cuando acabes asegurate de que tu primer PR sea aprobado primero luego cuando este en master hacer pull de master y hacer un merge de master a tu branch para que esten parejas, y checar que no haya conflicto con nada, una vez que master esta a la par con tu branch + los cambios de tu segunda branch checas que todo funcione y ya hacerforce pusha tu PR para que quede todo parejo.
Vale? Igual cualqueir cosa me avisas!
app/views/documents/index.html.erb
Outdated
| @@ -16,7 +16,8 @@ | |||
| <div class="card-content blue-grey-text center-align"> | |||
| <span class="card-title">Video 1: 6 tips de Salud 360</span> | |||
| <video style="width: 640px; height: 385px;" controls> | |||
| <source src="https://res.cloudinary.com/he98sycd1/video/upload/v1550518726/Salud_360.mp4" type="video/mp4"> | |||
| <source src="https://res.cloudinary.com/hkm7dxrkk/video/upload/v1553031151/salud360/Pro_Salud_30_no.2.mp4" type="video/mp4"> | |||
| <!-- Documentos <source src="https://res.cloudinary.com/he98sycd1/video/upload/v1550518726/Salud_360.mp4" type="video/mp4">--> | |||
There was a problem hiding this comment.
Este comment esta bien porque lo usaremos despues, pero igualrecordar que no dejamos codigo documentado, por ahorita esta bien
| @@ -25,7 +25,7 @@ | |||
| </div> | |||
| </div> | |||
|
|
|||
| <!-- Documentos | |||
| <!-- Documentos --> | |||
There was a problem hiding this comment.
Si se quedo como antes hay que borrar esos commits, igual con el rebase es facil borrar los commits para mantener el historial de commits limpio y no causar confusion, estuve 5 minutos tratando de ver que paso o porque lo quitaste haha hasta que vi el siguietne commit 😆
app/views/documents/index.html.erb
Outdated
| @@ -16,8 +16,7 @@ | |||
| <div class="card-content blue-grey-text center-align"> | |||
| <span class="card-title">Video 1: 6 tips de Salud 360</span> | |||
| <video style="width: 640px; height: 385px;" controls> | |||
| <source src="https://res.cloudinary.com/hkm7dxrkk/video/upload/v1553031151/salud360/Pro_Salud_30_no.2.mp4" type="video/mp4"> | |||
| <!-- Documentos <source src="https://res.cloudinary.com/he98sycd1/video/upload/v1550518726/Salud_360.mp4" type="video/mp4">--> | |||
| <source src="https://res.cloudinary.com/he98sycd1/video/upload/v1550518726/Salud_360.mp4" type="video/mp4"> | |||
There was a problem hiding this comment.
Este commit ni cuando lo comentaste deberian de estar, hay que borrarlos, hehehe
app/views/documents/index.html.erb
Outdated
| @@ -31,10 +30,10 @@ | |||
| <div class="col s12"> | |||
| <div class="card" style="width: 91%; margin: auto; text-align='center'"> | |||
| <div class="card-content blue-grey-text center-align"> | |||
| <span class="card-title"><%= document.title %></span> | |||
| <span class="card-title"><%= document.titulo %></span> | |||
There was a problem hiding this comment.
Porque cambiaste este? igual el attributo estaba como title, hiciste la migracion despues? A lo que me queda otra duda, es requerimiento que la programacion este en espaniol? es importante saber. No estaba enterado, para las siguietnes cosas hacerlas en espaniol. 👍
| <video style="width: 640px; height: 385px;" controls> | ||
| <source src="https://res.cloudinary.com/hkm7dxrkk/video/upload/v1553031151/salud360/Pro_Salud_30_no.2.mp4" type="video/mp4"> | ||
| </video> | ||
| <p>índice glucemico de los alimentos</p> | ||
| <p></p> |
There was a problem hiding this comment.
Si no contiene nada deberia de ser mejor solo quitarlo.
app/views/documents/index.html.erb
Outdated
| @@ -30,10 +30,10 @@ | |||
| <div class="col s12"> | |||
| <div class="card" style="width: 91%; margin: auto; text-align='center'"> | |||
| <div class="card-content blue-grey-text center-align"> | |||
| <span class="card-title"><%= document.titulo %></span> | |||
| <span class="card-title"><%= document.title %></span> | |||
There was a problem hiding this comment.
Lo mismo, clean commit history hehehe no tienes que commitear cada paso, sino el resultado final
|
Dado el mal manejo de los commit y el desarrollo de una propuesta, esta branch ya no es necesaria. |
#93
Se agrego el video número 2 que separa la fases de los documentos.