Does this functionality belong in the repo? about messenger-bot HOT 4 CLOSED

I've also added a function which allows the user to check if a given message was sent as part of a separateMessage. This is necessary if the user wants to implement their own on delivery listener, but doesn't want to handle those messages part of separateMessages which are delivered.

from messenger-bot.

@hpincket
Thanks for your contribution, but I have a few concerns:

I am not a javascript developer, but the implementation looks awful.
The main repo is kept in OOP paradigm, meaning that one can create more than one Bot instance.
As you can probably see registering multiple bot instances for your module does not work.

Moreover, I don't see a reason to send multiple messages, maybe for aesthetic purposes?
Also I don't personally think it fits here, but that's just my personal opinion of course so feedback from other devs is appreciated.

from messenger-bot.

@maciekmm
Thanks for the feedback.

As I'm sure you've picked up, I am no javascript developer myself. So I warmly welcome criticisms and suggestions. You're right, this will only work with a single bot, how short-sighted of me! Besides that, would you rather the implementation lie inside of the existing OOP Bot object? I am just trying to understand what a better implementation might look like.

Of course, if you don't see a fit in the repo, the above questions are solely for my improvement.

As far as a use case goes, I wrote a bot which needed to convey a paragraph or two of information to the user. We began with a single message approach, but it was too much. The unapproachable wall of text negated any benefit gained from a conversational UI. The other approach was have the user respond when read for the next part of the message. (Something like "Are you ready to continue?"). This felt too robotic, so we went with separate messages.

from messenger-bot.

This is probably not something we want to implement at this time or with this specific API, I'll therefore close this issue. Thank you for the suggestion and effort put into this issue.

from messenger-bot.