[BUG] Migration check is not correct

botpress / nlu

This repo contains every ML/NLU related code written by Botpress in the NodeJS environment. This includes the Botpress Standalone NLU Server.

22 stars 21 forks source link

[BUG] Migration check is not correct #109

Closed allardy closed 2 years ago

allardy commented 2 years ago

The migration is missing a check to see if it need to run or not. I always use TESTMIG_NEW=true and AUTO_MIGRATE=true when developing and my config is overwritten everytime I reboot the server in botpress.config.json.

If the nlu key is already configured in botpress.config.json, the migration should be skipped

franklevasseur commented 2 years ago

Hi @allardy,

This issue is unrelated to NLU-server. Please open issues on the correct repo in the future.

Also, this only affects branch next of Botpress so it's not released yet. For this reason, it should not be treated as a Github issue.

I'll close the issue for the reasons mentioned above.