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

Feature/task 6 #28

Closed
wants to merge 11 commits into from
Closed

Feature/task 6 #28

wants to merge 11 commits into from

Conversation

moaqz
Copy link
Collaborator

@moaqz moaqz commented Jan 19, 2023

🧾 Invoices App Pull Request

Descripcion ✏️

  • Agregue un hook para cambiar el theme de la aplicación.
  • Agregue el componente que utiliza el hook para cambiar el theme. (este componente es el que faltaba en el navbar)

Cambios visuales 🎨

N/A

Como testear? 🐛

Ejecutando los tests con npm run test 😅

Copy link
Collaborator

@JuaniSilva JuaniSilva left a comment

Choose a reason for hiding this comment

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

El archivo pnpm-lock.yaml no debería estar en el pr

@moaqz
Copy link
Collaborator Author

moaqz commented Jan 19, 2023

Ahí eliminé el archivo pnpm-lock.yaml

Copy link
Owner

@nsdonato nsdonato left a comment

Choose a reason for hiding this comment

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

Se podría agregar un test que verifique algo de la aplicación que tenga que tener el color correspondiente y no solo el local storage?

@nsdonato nsdonato added the nueva función Nuevo feature o pedido label Jan 23, 2023
@nsdonato nsdonato linked an issue Jan 23, 2023 that may be closed by this pull request
3 tasks
src/components/Darkmode/index.tsx Show resolved Hide resolved
src/components/Darkmode/index.tsx Outdated Show resolved Hide resolved
src/hooks/useTheme/index.tsx Show resolved Hide resolved
@moaqz
Copy link
Collaborator Author

moaqz commented Jan 23, 2023

Borre el package.json viendo el problema de las dependencias sin querer y tuve que hacer un git force 😨 (seguro hay una mejor forma). De todos modos tengo problemas con el package.json y las dependencias porque instale @testing-library/react-hooks para testear el hook y había un problema con las peer dependencies.

@@ -0,0 +1,10 @@
import Darkmode from ".";
Copy link
Collaborator

@phreakphreak phreakphreak Jan 24, 2023

Choose a reason for hiding this comment

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

mover este test a la siguiente ubicacion src/tests/DarkMode/DarkMode.test.tsx

src/hooks/useTheme/index.tsx Show resolved Hide resolved
src/components/Darkmode/index.tsx Outdated Show resolved Hide resolved
@nsdonato nsdonato closed this Jan 25, 2023
@nsdonato nsdonato deleted the feature/task-6 branch January 25, 2023 03:40
@nsdonato nsdonato restored the feature/task-6 branch January 27, 2023 01:10
@nsdonato nsdonato deleted the feature/task-6 branch January 27, 2023 01:12
@nsdonato
Copy link
Owner

@techwithmat no te hagas problema, yo resolví el problema de las dependencias, ya esta este pr en main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
nueva función Nuevo feature o pedido
Projects
Status: Hecho
Development

Successfully merging this pull request may close these issues.

Agregar Light/Dark mode
4 participants