chatbase-node icon indicating copy to clipboard operation
chatbase-node copied to clipboard

Improveable design

Open sueess opened this issue 8 years ago • 0 comments

Many thanks for the useful library. I just integrated Chatbase with this library. In my eyes there are already some design issues with this library:

  • Neither the Chatbase documentation nor this library really show, which properties are valid for a user- or an agent-message. E.g. does Chatbase concern about the intent-property of an agent message? Would be great if this could be documented more clearly and reflected in the library.

  • setHandled(boolean) would be more handy than two methods setAsNotHandled/setAsHandled. Like this it needs usually an additional condition in the code.

  • What's the idea behind the set.newMessage-method. Why not offering the same interface that set usually has: set.add(new Message());?

  • Are there any Typescript typings?

I'm curious about your thoughts.

sueess avatar Dec 08 '17 12:12 sueess