Skip to content

Gx 23806 elements buttons #388

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

Merged
merged 3 commits into from
Jul 17, 2023
Merged

Gx 23806 elements buttons #388

merged 3 commits into from
Jul 17, 2023

Conversation

Braian1396
Copy link

Los tres casos de prueba fueron exitosos y se pudo comprobar que al realizar cada una de las acciones correspondientes, se mostro el mensaje esperado.

@Braian1396 Braian1396 added Needs Review Ready for review by the team. Tutor Assigned I have a tutor assigned labels Jul 13, 2023
@Braian1396 Braian1396 requested a review from ZwitterKaneda July 13, 2023 13:11
@Braian1396 Braian1396 self-assigned this Jul 13, 2023
@ZwitterKaneda ZwitterKaneda added In Review Currently being evaluated by the reviewer. and removed Needs Review Ready for review by the team. labels Jul 16, 2023
@ZwitterKaneda
Copy link
Contributor

hola @Braian1396
Mira por aquí te dejo la solución a porque se te dañó el test 😅
image
Este lodash se cargará cada vez que guardes un cambio en un test con una sintaxis que no reconoce, en este caso en algún momento guardaste el eq sin el . o los parentesis y cargó esa linea para intentar resolver el conflicto.
Los espacios extras del test, ya que estas realizando cambios, borralos para que sea más cómodo de leer y cuente con menor cantidad de lineas ;)

Recuerda: Tienes que hacer fetch de la rama QA para actualizarla para poder integrar los cambios que se fueron realizando. Los pasos serán:
git checkout GX-23806-elementsButtons
git fetch origin QA
git merge origin/QA
Aquí debería de tener un conflicto con tu archivo CI-Suite.yml ya que te llevarás el que esta en QA y el tuyo tiene otro cambio, debes de aceptar tu cambio.

@ZwitterKaneda ZwitterKaneda added Requirements Not Met Does not meet specified requirements. and removed In Review Currently being evaluated by the reviewer. labels Jul 16, 2023
@ZwitterKaneda ZwitterKaneda added Approved Approved and pending merge. and removed Requirements Not Met Does not meet specified requirements. labels Jul 17, 2023
@ZwitterKaneda
Copy link
Contributor

Excelente trabajo @Braian1396 continuemos! 🚀

@ZwitterKaneda ZwitterKaneda merged commit b0cf70b into QA Jul 17, 2023
@AlexanderWindecker AlexanderWindecker deleted the GX-23806-elementsButtons branch August 26, 2024 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved Approved and pending merge. Tutor Assigned I have a tutor assigned
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants