[WEB-MAD DE] Lali Osorio#2436

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Closed

laliiosorio wants to merge2 commits intoironhack-labs:masterfromLabs-IronHack:master

Closed

[WEB-MAD DE] Lali Osorio#2436

laliiosorio wants to merge2 commits intoironhack-labs:masterfromLabs-IronHack:master

Conversation

Copy link

laliiosorio commentedJun 3, 2021

Duda infinita del ejercicio Iteration#6: Find elements, me funcionaba perfecto cuando la palabra existia me salia true y cuando no undifined fue colocarle el else false y nunca mas volvio a ser true solo con index 0. Al final le di tantas vueltas que ni sé cual versión quedo jajajajaj Vi algo de falsy pero ni idea. también la coloque en una variable para colocar un solo return pero se quedaba el false para siempre. No sé me ocurrieron otras maneras. Anyways, ya me pegaré con los BONUS a ver qué explota!!!

PD: Mi PC peto cuando le di npm run test:watch. Me toca npm run test y ya para que no colapse.

PD2: Me vine arriba con este pull request! Jajajaja 🤯

laliiosorio added2 commits

June 3, 2021 20:45

some functions OK

6a9c734

Finish

a704186

Copy link

Contributor

ta-web-mad commentedJun 4, 2021

Hola Lali!
Venirse arriba con las pull request esta genial para poder ayudarte así que tú muy arriba siempre!
Lo del test:watch...no sé como ayudarte, podríamos mirarlo pero va a ser tema del ordenador, lo vemos luego si quieres.
En cuanto a tu duda con la iteración 6:
Veo que te quedaste probando pero la idea era buena:

Lo que pasa que te liaste con el bucle y con la solución que habías encontrado no lo necesitabas:

Simplemente con el includes, si le devuelves el true o false dependiendo de si el array contiene la palabra o no ya estaria! CASI!
Buen trabajo! Javascript no se te da mal como tú decias!

Copy link

stalebot commentedJul 8, 2021

This pull request has been automatically marked as stale because it didn't have any recent activity. It will be closed if no further activity occurs. Thank you for your contributions.