Corregido Storybook del componente Text #18
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ated its imports on Storybooks
pablonortiz
requested review from
colomfernando,
christian97dd,
pablodamianberdun,
dam788,
Maukr,
WilliamSaya-lvl30 and
GonzaFran
dam788
approved these changes
Aug 14, 2023
Pull Request Test Coverage Report for Build 5870192674
💛 - Coveralls |
WilliamSaya-lvl30
requested changes
Aug 15, 2023
src/components/Text/index.tsx
Outdated
|
|
||
| interface TextProps extends TextComponentProps { | ||
| children?: ReactElement | string; | ||
| style?: TextStyle[]; |
There was a problem hiding this comment.
Creo que no es necesario pasar un array de estilos, de esta manera creo que se ve mejor, y no hay que desectructurar style.
cuando invocas un componente no se le debería pasar mas de objeto de 1 estilo.
src/components/StatusChip/index.tsx
Outdated
| @@ -19,7 +20,7 @@ const StatusChip = ({children, ...props}: StatusChipProps) => { | |||
|
|
|||
| return ( | |||
| <View style={styles(props).ViewStyles} {...props}> | |||
| {isCustomComponent ? children : <Text style={styles(props).TextStyles}>{children}</Text>} | |||
| {isCustomComponent ? children : <Text style={[styles(props).TextStyles]}>{children}</Text>} | |||
There was a problem hiding this comment.
Esto tendria que ver con el otro comentario, aqui no hace falta pasar los estilos como array
There was a problem hiding this comment.
Listo @WilliamSaya-lvl30 ahí ya lo corregí!
WilliamSaya-lvl30
approved these changes
Aug 17, 2023
christian97dd
approved these changes
Aug 17, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
LINK DE TICKET:
https://janiscommerce.atlassian.net/browse/JUIP-113
DESCRIPCIÓN DEL REQUERIMIENTO:
Contexto
Actualmente se está tomando el componente Text de un archivo que se está armando en una carpeta del mismo nombre en la carpeta de Storybook
Necesidad
Se necesita usar el componente Text de la carpeta components, eliminando el archivo en dónde se arma el Text que se está utilizando actualmente en Storybook
DESCRIPCIÓN DE LA SOLUCIÓN:
Se eliminó el componente Text que se estaba utilizando en Storybook, por esto, se modificaron los imports del Storybook Text y StatusChip para utilizar el componente Text de la carpeta components.
Además, se modificó el componente Text, esto debido a que no estaba aceptando los props de styles correctamente, por lo que ahora se le comenzaron a pasar el resto de props que se le pasen al componente, al StyleSheet del componente.
CÓMO SE PUEDE PROBAR?
Clonar e instalar el Repo de UI-Native
Correr los siguientes 3 comandos en tres pestañas diferentes de la termina
npm run start
npm run android
npm run storybook
Se debe abrir la app en el emulador con storybooks y el navegador en la url http://localhost:7007/ donde se podrá probar
el comportamiento del componente
Para 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