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

Botpress X support #5

Open
Japh opened this issue Mar 14, 2018 · 7 comments
Open

Botpress X support #5

Japh opened this issue Mar 14, 2018 · 7 comments

Comments

@Japh
Copy link

Japh commented Mar 14, 2018

Are you planning to update to support Botpress X? Any estimated timeframe?

@MagnificentPako
Copy link
Owner

I'm not really sure what Botpress X is, tbh. I've been "out of the loop" for some time now but I'll definitely take a look, considering that people actually use this, or at least intend to.

@Japh
Copy link
Author

Japh commented Mar 14, 2018

It's the upcoming major version update, that breaks backwards compatibility, but is in beta at the moment.

I've reverted back to the Botpress stable version so I can use this module.

@MagnificentPako
Copy link
Owner

Oh, I'll definitely take a look now then.

As a user, would you prefer I keep the function names as they are or revert some of my idiocy and make them look more "standard"? (I believe i messed up the text sending one and the event for text messages which, I believe, makes it harder to interact with).

@MagnificentPako
Copy link
Owner

I'll unfortunately have to put this on hold until the core developers decide to make Botpress X actually work. I tried to run it using NodeJS 9.8 which resulted in it not even installing. Then I downgraded to 8.10 and I was greeted with a new message telling me that the generated bot only runs on versions up to 8.9. Because I couldn't bother downgrading again I decided to change the requirements in the package.json instead. When trying to run the bot with yarn start I'm greeted with a multitude of errors. Even after following the acclaimed fix from Luiz Carvalho which just shifted it to another error message and the inability to initialize new bots.

I sincerely hope they fix this :/

@Japh
Copy link
Author

Japh commented Mar 14, 2018

As a user, would you prefer I keep the function names as they are or revert some of my idiocy and make them look more "standard"? (I believe i messed up the text sending one and the event for text messages which, I believe, makes it harder to interact with).

Standardising would be better, and yeah I had to hack the text sending one because it was sending an embed all the time, no checks.

I'll unfortunately have to put this on hold until the core developers decide to make Botpress X actually work. I tried to run it using NodeJS 9.8 which resulted in it not even installing. Then I downgraded to 8.10 and I was greeted with a new message telling me that the generated bot only runs on versions up to 8.9. Because I couldn't bother downgrading again I decided to change the requirements in the package.json instead. When trying to run the bot with yarn start I'm greeted with a multitude of errors. Even after following the acclaimed fix from Luiz Carvalho which just shifted it to another error message and the inability to initialize new bots.

Sad to hear that... I too hope they fix it. I imagine they will before release.

It was working fine for me on LTS. I just use nvm to install whatever Node version is needed for a project.

@PilouFaace
Copy link

Hey ! Now Botpress X is stable . So, are you planning to update to support Botpress X? Any estimated timeframe?

@MagnificentPako
Copy link
Owner

@PilouFaace hey, sorry for the late reply. Botpress changed a lot from when I first started using it and I unfortunately don't have the time to re-learn it; University is occupying a lot of my energy. If I ever get some free time I'll definitely take a look again, although I can't promise anything.

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

3 participants