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

Añadir compatibilidad de componentes con styled components #20

Conversation

pablonortiz
Copy link
Contributor

@pablonortiz pablonortiz commented Aug 16, 2023

LINK DE TICKET:
https://janiscommerce.atlassian.net/browse/JUIP-115

DESCRIPCIÓN DEL REQUERIMIENTO:

Contexto

Actualmente, al intentar utilizar el componente con styled components, al ser vinculado a alguna de las tres apps, esto no es posible

Necesidad

Se requiere hacer los componentes, que sean necesarios, compatibles para ser estilizados con styled components

DESCRIPCIÓN DE LA SOLUCIÓN:

Se le agregó la prop style a los componentes que sea posible editar los estilos para que, de esta manera, se los pueda estilizar con styled components

CÓMO SE PUEDE PROBAR?
Clonar e instalar el Repo de UI-Native

Linkear el componente con un proyecto, seguir la siguiente documentación
https://fizzmod.atlassian.net/wiki/spaces/JAPP/pages/2341765125/C+mo+trabajo+con+el+package+UI

Luego, se deberá probar que los distintos componentes modificados sean compatibles para editar con styled components

@coveralls
Copy link

coveralls commented Aug 16, 2023

Pull Request Test Coverage Report for Build 5883190187

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 5878399436: 0.0%
Covered Lines: 154
Relevant Lines: 154

💛 - Coveralls

Copy link
Contributor

@pablodamianberdun pablodamianberdun left a comment

Choose a reason for hiding this comment

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

Para el componente input, cambiar la inteface a TextStyle

@pablodamianberdun pablodamianberdun merged commit 4854e7f into master Aug 16, 2023
2 checks passed
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.

None yet

5 participants