Skip to content

Gx 40164 ✅ tools qa elements buttons #508

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 4 commits into from
Oct 21, 2023

Conversation

Franciscomedina240792
Copy link

Se realizó la cobertura de Casos de prueba que fueron tres funcionalidades de botones. Se validó el correcto funcionamiento de dichos botones que son "Double Click Me", "Right Click Me" y "Click Me", donde se verificó que al realizar la acción mencionada en cada uno de ellos, apareciera el mensaje correspondiente

@Franciscomedina240792 Franciscomedina240792 added Approved Approved and pending merge. Tutor Assigned I have a tutor assigned labels Oct 20, 2023
@Franciscomedina240792 Franciscomedina240792 self-assigned this Oct 20, 2023
@yeyu2083 yeyu2083 added In Review Currently being evaluated by the reviewer. and removed Approved Approved and pending merge. labels Oct 20, 2023
@yeyu2083
Copy link

Hola @Franciscomedina240792 ,

Quiero comenzar diciendo que has hecho un buen trabajo en esta primera User Story (US). Me agrada que te retes a ti mismo y busques mejorar continuamente.

Sin embargo, me gustaría señalar un par de puntos que podrían hacer que tu trabajo sea aún mejor:

Mensaje en el último commit: Sería útil que en el futuro, dejes un mensaje en el último commit para que podamos entender mejor los cambios que realizaste. Esto facilita la colaboración y la revisión del código para todos los miembros del equipo.

Código comentado: Generalmente, no solemos subir el código con comentarios, a menos que sea absolutamente necesario o aporte un valor significativo. Los comentarios pueden dificultar la lectura del código y pueden volverse obsoletos con el tiempo. Te animo a que, en lugar de comentarios, escribas un código claro y legible que se explique por sí mismo. Si hay secciones particularmente complicadas o decisiones de diseño importantes, podríamos considerar agregar comentarios en esos casos.

¡Ánimo! Continuemos avanzando juntos.

@yeyu2083 yeyu2083 added Approved Approved and pending merge. and removed In Review Currently being evaluated by the reviewer. labels Oct 20, 2023
@yeyu2083 yeyu2083 merged commit 071d559 into QA Oct 21, 2023
@AlexanderWindecker AlexanderWindecker deleted the GX-40164-✅-tools-qa-elements-buttons branch August 26, 2024 23:31
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