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

Appsrn 264 documentacion de package y GitHub actions #3

Merged

Conversation

GonzaFran
Copy link
Contributor

LINK DE TICKET: *

https://janiscommerce.atlassian.net/browse/APPSRN-257

LINK DE SUBTAREA:

https://janiscommerce.atlassian.net/browse/APPSRN-264

DESCRIPCIÓN DEL REQUERIMIENTO: *

Se necesita agregar la documentación sobre el package y los metodos y componentes

DESCRIPCIÓN DE LA SOLUCIÓN: *

Se agregó la documentación sobre los metodos y componentes necesarios del package.
También se agregaron las githubActions para realizar un control de coverage del package, documentación y su posterior publicación.

Se agregaron las firmas en jsdoc de useNotificacion y notificationProvider.

Se agregaron los scripts utilzados en analytics para enviar la notificación a slack sobre una nueva publicación del package

ACLARACIÓN:

No se agregó la documentación de topicsSuscription y topicsUnsuscription porque, como mencionamos en el PR anterior, estas funciones no serán utilizadas a futuro; cuando back termine el trabajo de su lado nosotros utilizaremos otros metodos para suscribir al usuario a eventos y estas utils serán deprecadas.

Tampoco incluí la documentación de utils dentro del readme ya que estas no podrán ser importadas desde el package

@christian97dd christian97dd merged commit 2c5ba1b into main Jan 15, 2024
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.

3 participants