Skip to content
This repository has been archived by the owner on Jun 27, 2022. It is now read-only.

New PersonItem component. #17

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

New PersonItem component. #17

wants to merge 3 commits into from

Conversation

Vallejoanderson
Copy link
Collaborator

@Vallejoanderson Vallejoanderson commented Mar 15, 2022

image

@vercel
Copy link

vercel bot commented Mar 15, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/juanvqz/swapi/2hWx8qTF4Zd4aV5Tzcf4PGUqrjw4
✅ Preview: https://swapi-git-personitem-juanvqz.vercel.app

src/pages/people.js Outdated Show resolved Hide resolved
src/pages/people.js Outdated Show resolved Hide resolved
src/pages/people.js Outdated Show resolved Hide resolved
src/pages/people.js Outdated Show resolved Hide resolved
src/pages/people.js Outdated Show resolved Hide resolved
src/components/PersonItem.js Outdated Show resolved Hide resolved
@JuanVqz
Copy link
Owner

JuanVqz commented Mar 16, 2022

somos señores no seniors 🤣

@davbrito
Copy link
Collaborator

somos señores no seniors 🤣

🤣🤣🤣

@JuanVqz
Copy link
Owner

JuanVqz commented Mar 28, 2022

@Vallejoanderson cuando tengas oportunidad estaría bueno que corrijas los comentarios que David hizo, por fas, para darle merge 👍

Co-authored-by: David Brito <39559632+davbrito@users.noreply.github.com>
@Vallejoanderson
Copy link
Collaborator Author

Creo que ya quedaron aprobados todos los cambios gente, muchas gracias por la paciencia! 🙌

@JuanVqz
Copy link
Owner

JuanVqz commented Apr 8, 2022

@Vallejoanderson Ahora el linter se esta quejando, podrías arreglarlo por favor

@davbrito
Copy link
Collaborator

davbrito commented Apr 9, 2022

Es por el prettier. Tienes que formatear los archivos que editaste

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants