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

Database migration to manage app database #196

Open
DanSilvaDev opened this issue May 16, 2019 · 3 comments
Open

Database migration to manage app database #196

DanSilvaDev opened this issue May 16, 2019 · 3 comments

Comments

@DanSilvaDev
Copy link

DanSilvaDev commented May 16, 2019

I got some troubles trying to run the RasaUI due database scripts. As they're splited and even the main one has some columns missing.

I developed the migration feature with db-migrate, now when run npm start the app executes the migration and then connect to the database. The user won't have to worry about lot of sql files. The developers just use the db-migrate create migrationName and add or change the database.

It simplifies and automate the process on both sides.

I know that I should have raised this issue before, but as I was really in need of this feature I developed already in my fork and I'll send the pull request, too. I do think that it's very useful and make the Rasa UI more robust.

Any doubts or further clarifications, let me know.

Glad to help .

Marking for review:
@paschmann
@pradeepmvn

@JoseZamora97
Copy link

hola @ProgramadorDeValor se que estarás muy ocupado, me podrias ayudar con un problema que tengo de rasa ui?

@DanSilvaDev
Copy link
Author

Hola @JoseZamora97,

No sé si puedo ayudarte, porque no soy un experto en la herramienta. Pero de todos modos, podemos intentarlo así: Abra un "issue" en ese proyecto describiendo detalladamente el problema. Si lo sabes te ayudo, sin problemas. Si no lo sabe, quedará abierto para el apoyo de la comunidad.

Como no hablo español, he usado el traductor. Me disculpa si hubo algún error :)

@JoseZamora97
Copy link

Hi @ProgramadorDeValor here is my issue #192 for some reason I am getting a nlu error when I test the trained model. Thanks for reply C:

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

No branches or pull requests

2 participants