Skip to content

[BCN-PT-082022] - Juan Carlos Pineda - - #16

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

[BCN-PT-082022] - Juan Carlos Pineda - #16
laggart wants to merge 2 commits into
Conforcat-Frontend-PT-Jul:masterfrom
laggart:master

Conversation

@laggart

@laggart laggart commented Aug 9, 2022

Copy link
Copy Markdown

Names and Inputs realizado JS.

@Conforcat-ta-webdev

Copy link
Copy Markdown

Hoola Juan Carlos! 👋

Recuerda por favor poner en el campus el enlace correcto de la pull y no del repo 😊 Para la próxima! jajaja
Te comento un par de cosas que he visto en el lab:

  • Los console.log, no están mal pero sería genial que lo hicieses mediante interpolación, esto es la legible y te permitirá hacer más cosas en el futuro. Quedaría algo como esto:
const hacker1 = "Name"
console.log(`My name is ${hacker1}`)
  • Código limpio y entendible ✅

  • Mencionarte que en la iteración 3 donde nos piden ordenar alfabéticamente, hay que tener en cuenta las mayusculas y las minúsculas, pues pueden afectar al resultado. Para solucionarlo únicamente hay que indicar un .toLowerCase() a nuestros hackers 😊

Te animo a que en los siguientes le des caña también a los bonus que siempre se aprende algo!

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