Skip to content

Conversation

@Nataniel4
Copy link
Contributor

PAQUETE DATABASE-DISPATCHER
JCN-68 Paquete npm database-dispatcher

LINK AL TICKET
https://fizzmod.atlassian.net/browse/JCN-68

DESCRIPCIÓN DEL REQUERIMIENTO
Se solicita el desarrollo del database-dispatcher en paquete npm.

DESCRIPCIÓN DE LA SOLUCIÓN
Se desarrollo el paquete npm de database-dispatcher.

@gastonpereyra
Copy link
Contributor

getDatabase No carga correctamente los modulos.

@gastonpereyra
Copy link
Contributor

gastonpereyra commented May 29, 2019

README.md

  • Dice databaseConfig() debería decir databaseConfig.
  • Dice que el error se maneja con SchemaValidatorError debería decir DatabaseDispatcherError

Copy link
Contributor

@gastonpereyra gastonpereyra left a comment

Choose a reason for hiding this comment

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

Me funciono bien luego del cambio en el método donde requería los drivers de las bases de datos.
Comentario:
Los drivers de Janis de las bases de datos, si no van a llevar el '@janiscommerce' en el nombre, habría que modificar el nombre en el database-dispatcher

@juanhapes juanhapes requested a review from gastonpereyra May 29, 2019 20:00
@juanhapes juanhapes self-assigned this May 29, 2019
@gastonpereyra gastonpereyra self-requested a review June 10, 2019 15:08
@juanhapes juanhapes merged commit 2138148 into master Jun 13, 2019
@Nataniel4 Nataniel4 deleted the JCN-68-database-dispatcher branch June 13, 2019 13:51
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.

5 participants