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

agregar componente lista #28

Merged
merged 2 commits into from Nov 29, 2023

Conversation

dam788
Copy link

@dam788 dam788 commented Nov 21, 2023

LINK DE TICKET:

https://janiscommerce.atlassian.net/browse/JUIP-131

DESCRIPCIÓN DEL REQUERIMIENTO:

La idea de este componente es que el mismo pueda renderizar el array que se le pase y, a partir de una prop, determinar si se estará renderizando en una FlatList o en un ScrollView, esto para poder dar más versatilidad al componente y poder emplearlo en más lugares

DESCRIPCIÓN DE LA SOLUCIÓN:

  • Se unificaron FlatList y Scrollview en un solo componente y hereda todas las props de los componentes de lista.
Prop Default Requerido Tipo Obs.
data null true array: string, mumber, object puede ser un array de objetos, string o number.
type null false enum: flatList, scrollView hereda las props de los componentes de lista.
renderComponent null true React.ReactElement recibe por parametros la info de data que es utilizado para pasarle el componente de forma custom.

CÓMO SE PUEDE PROBAR?

  1. Entramos a la rama JUIP-132-agregar-componente-lista-a-ui-native en @janis-commerce/ui-native
  2. En consola tiramos yalc push && yalc publish
  3. En el repo deen cualquier rama que usemos para probar tiramos yalc add @janiscommerce/oauth-native && npm i
  4. Reemplazar cualquier componente de lista y realizar pruebas de estilos o pasarle props de de Scrollview o FlatList según lo que seleccione...

SCREENSHOTS:

DATOS EXTRA A TENER EN CUENTA:

Si bien pudieron heredarse todas las props de los componentes, como no hay formas de separarlas según lo que uno renderice tuvieron que ser fusionadas

@coveralls
Copy link

Pull Request Test Coverage Report for Build 6945237721

  • 7 of 7 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 6798700218: 0.0%
Covered Lines: 357
Relevant Lines: 357

💛 - Coveralls

@christian97dd christian97dd merged commit 4165c53 into master Nov 29, 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