Skip to content

Entry challenge#14

Open
blindlp wants to merge 17 commits into
corecodeio:masterfrom
blindlp:entry-challenge
Open

Entry challenge#14
blindlp wants to merge 17 commits into
corecodeio:masterfrom
blindlp:entry-challenge

Conversation

@blindlp
Copy link
Copy Markdown

@blindlp blindlp commented Jun 20, 2020

Hola he tratado de realizar lo solicitado, con lo cual les adjunto el trabajo realizado para su revision y verificación.

mi correo es lpozuelos@irc.com.gt

Saludos

Copy link
Copy Markdown

@netpoe netpoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@blindlp recibí tu correo, pero la claveDeAcceso es incorrecta.
Por favor revisa la respuesta del servidor después del GET.

Como lo indican las instrucciones

@areyes107
Copy link
Copy Markdown

Muy buen comienzo @blindlp , revisa los comentarios a continuación

Comment thread src/index.ts Outdated
Comment thread src/profile.json Outdated
Comment thread src/postProfile.ts
Comment thread src/getProfile.ts
“blindlp” added 2 commits June 22, 2020 20:43
Copy link
Copy Markdown

@netpoe netpoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@blindlp resuelve los comentarios.
Ya instalaste prettier y la configuración de settings.json que te sugerí?

@areyes107 recuerda marcar como Resolve Conversation los cambios que ya se hayan realizado.

Comment thread src/index.ts
Comment thread src/postProfile.ts
@blindlp blindlp requested a review from netpoe June 23, 2020 20:29
@blindlp
Copy link
Copy Markdown
Author

blindlp commented Jun 23, 2020

Listo los cambios fueron aplicados.

Copy link
Copy Markdown

@netpoe netpoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@blindlp buena organización de tus funciones. 👍

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.

3 participants