-
Notifications
You must be signed in to change notification settings - Fork 11
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
Implement missing chatbot functionalities #4
Comments
The close button functionality (assuming you refer to the X in the top right of the "WordPress Assistant") could just collapse the container or it could it clear the chat window. It could persist by default but then there could be ways to clear it - settings for closing it on collapse or a clear chat button. How you envision that part of the interface? Points 1 and 3 are closely related and connected to a wider user meta / session feature to build out like #5 . At first it can probably just clear when they logout (or some other action) or would there be saved chats with an interface to manage those as well? |
My idea was simply to close the interface. The only reason I haven't implemented it yet is that the
The difference I see in point 1 and 3 is that point 1 is more important to be reliable than point 3. In other words, I'm thinking to implement chat history persistence on the server-side (e.g. relying on user meta) whereas whether the chat window is open is a simple non-destructive UI preference that's easiest managed clientside, via something like You mention #5, however where I am thinking to treat this differently is that I consider #5 a functionality to build into the underlying AI infrastructure, whereas the whole chatbot is essentially a user-facing feature built on top of that AI infrastructure. So I'd like for managing the chat history persistence to happen as part of that specific feature - since I don't want to mandate how any kind of chat feature another plugin may implement should or should not implement its persistence. Regarding chat history clearing, I was thinking that it would be persistent beyond sessions, e.g. timestamp based where we would automatically remove messages older than a certain point, let's just say for example 1 week old. But it's worth thinking through the possible options. Maybe it would even be enough to only store these messages client-side for the session? 🤔 Let me know if that makes sense, or any other thoughts you may have. |
For now, the history is persisted using a simple solution with A proper persistence layer that uses the WordPress database and therefore could persist beyond sessions could be explored in the future. |
The one user-facing demo feature in the plugin, the chatbot, is not fully implemented yet. A few crucial pieces are missing:
The text was updated successfully, but these errors were encountered: