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

added baseCollapsible component #48

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

GonzaFran
Copy link
Contributor

@GonzaFran GonzaFran commented Dec 3, 2024

LINK DE TICKET:
https://janiscommerce.atlassian.net/browse/JPRN-1995

CONTEXTO:

En el desarrollo que vamos a trabajar en la app de orders (https://janiscommerce.atlassian.net/browse/ATR-1427) necesitamos usar colapsables para rediseñar los canastos que pertenecen a cada pedido de una ronda, y, ya que en todos los repos de apps se utiliza el componente Collapsible como esqueleto para trabajar cada colapsable dentro de la aplicación, se decidió que sería más práctico tener el componente dentro de ui-native para facilitar su utilización en cada caso.

NECESIDAD:

Agregar el componente Collapsible en ui-native

SOLUCIÓN:

https://www.npmjs.com/package/react-native-collapsible

Se implementaron recursos y hooks de la librería reanimated para armar un componente colapsable nativo básico que reciba un componente header y un componente content.
El colapsable se encarga de renderizar ambos componentes y mostrar el header de cara al usuario, si el usuario quiere ver el contenido deberá presionar el Header, que está wrappeado dentro de un componente con propiedades de botón que permiten manipular la vista en pantalla.
Se puede modificar la duración de la animación de despliegue agregando la prop duration que espera recibir un tiempo representado en milisegundos.

¿CÓMO PROBARLO?
Puede vincular el pkg al repo que quiera e importar el componente Collapsible en el repo, y deberá agregar los 2 componentes requeridos.

En caso de tener un problema similar a este al tratar de levantar la aplicación

Error: Command failed: ./gradlew app:installBetaDebug -PreactNativeDevServerPort=8081
error Unable to resolve module atoms/SwipeUp/childComponents from /var/www/janis-picking-app/.yalc/@janiscommerce/ui-native/dist/index.js: atoms/SwipeUp/childComponents could not be found within the project or in these directories:
  node_modules
  11 | import Svg from 'atoms/Svg';
  12 | import SwipeUp from 'atoms/SwipeUp';
> 13 | import { SwipeUpFlatList, SwipeUpScrollView, SwipeUpView } from 'atoms/SwipeUp/childComponents';
     |                                                                  ^
  14 | import Text from 'atoms/Text';
  15 | import BaseInput from 'atoms/BaseInput';
  16 | import BaseCollapsible from 'atoms/BaseCollapsible';

FAILURE: Build completed with 2 failures.

1: Task failed with an exception.
-----------
* What went wrong:
Execution failed for task ':app:createBundleBetaDebugJsAndAssets'.
> Process 'command 'node'' finished with non-zero exit value 1

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

Lo que debe hacer, dentro del pkg de ui-native, es correr el comando npm run build

También puede ver el funcionamiento desde storybooks.

Peek 2025-01-08 10-58

Peek 2025-01-08 10-59

Copy link
Contributor

@pablonortiz pablonortiz left a comment

Choose a reason for hiding this comment

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

Lo estuve probando en la app de Delivery, storybook y storybook web y lo vi bien!

@GonzaFran GonzaFran requested a review from pablonortiz January 8, 2025 14:19
@coveralls
Copy link

coveralls commented Jan 8, 2025

Pull Request Test Coverage Report for Build 12679081991

Details

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

Totals Coverage Status
Change from base Build 12583349929: 0.0%
Covered Lines: 696
Relevant Lines: 696

💛 - Coveralls

Copy link
Contributor

@pablonortiz pablonortiz left a comment

Choose a reason for hiding this comment

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

Lo estuve probando y lo vi joya, lo único que estaría faltando es pullearte master y resolver unos conflictos, con eso ya estaría!

@pablonortiz pablonortiz self-requested a review January 9, 2025 13:20
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.

3 participants