Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR semana 6 #42

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

chapurestrepo
Copy link

## Ejercicio practico para repasar la creación de compoenentes en react en el bootcamp de undefined shell

  • Me parecio mas ordenado crear una carpeta para meter cada componente
  • Cree un archivo para alojar los arrays con la información necesaria para la navegación, los filtros y el contenido de las cards
  • En el archivo App.jsx compuse todo para llamar todos los componentes, mapear los datos para pintarlo

A modo personal he intentado practicar arrow functions, map, y bases de JS de las semanas anteriores

@socket-security
Copy link

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
eslint-plugin-react-refresh 0.4.3 None +0 11.6 kB arnaud-barre
eslint-plugin-react 7.33.1 None +0 794 kB ljharb

@chapurestrepo
Copy link
Author

que no se cierre el PR

@chapurestrepo
Copy link
Author

Que no caduque el pr

Copy link
Contributor

@jorge9372 jorge9372 left a comment

Choose a reason for hiding this comment

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

Saludos, tienes mi approve.

Un diseño sencillo, pero bastante agradable. Te recomiendo agregues algo de interactividad así sea algo sencillo para ir familiarizándote con eso.

Veo que organizas los componentes de otra manera, pero nada que criticar, cada quien tiene sus gustos y su forma de organizar su entorno de trabajo y es parte de lo bonito de todo este mundo no una sola manera sino cada quien con su estilo.

Felicidades sigue avanzando.

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