MagnificentPako / botpress-discord

GNU General Public License v3.0
11 stars 5 forks source link

Botpress X support #5

Open Japh opened 6 years ago

Japh commented 6 years ago

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

MagnificentPako commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 4 years ago

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

MagnificentPako commented 4 years ago

@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.