Skip to content

[BCN-PT-072022] - Daniel Vilches - #17

Open
muvildan wants to merge 2 commits into
Conforcat-Frontend-PT-Jul:masterfrom
danielvilches-Conforcat-Frontend-PT-Jul:master
Open

[BCN-PT-072022] - Daniel Vilches#17
muvildan wants to merge 2 commits into
Conforcat-Frontend-PT-Jul:masterfrom
danielvilches-Conforcat-Frontend-PT-Jul:master

Conversation

@muvildan

@muvildan muvildan commented Aug 9, 2022

Copy link
Copy Markdown

Iterations 1, 2, 3 + Bonus 1

@muvildan muvildan changed the title [BCN-PT-082022] - Daniel Vilches [BCN-PT-072022] - Daniel Vilches Aug 9, 2022
@Conforcat-ta-webdev

Copy link
Copy Markdown

Hoola Daniel! 🧑🏻‍🚀

Esta genial el lab! me encanta lo limpio que has hecho el codigo! 🚀

  • Muy buen uso de los métodos de arrays ✅
  • Código muy limpio ✅
  • En la iteración 3 donde hay que ver el orden alfabético, tienes que tener en cuenta las mayusculas, ya que pueden afectar al resultado. Con un .toLowerCase() en los nombres para partir desde los mismo valores lo solucionamos 😉

Esta muy bien!! sigue así! 🔥

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