- Issue created by @narendraR
- ๐ฎ๐ณIndia narendraR Jaipur, India
narendrar โ changed the visibility of the branch 3531955-ai-created-new-components to hidden.
- ๐ฎ๐ณIndia utkarsh_33
I tested the MR and here are my observations from it:-
- The history of messages remains intact even after navigating to the other page.
- The issue with this is that even after refreshing or closing the chatbot the history still remains intact.Does it makes sense to add a
Clear chat
button to reset history?
Rest everything looks good apart from small nits.Marking it NW for small changes.
- ๐ฎ๐ณIndia narendraR Jaipur, India
Added an issue for 'Clear chat' feature https://www.drupal.org/project/experience_builder/issues/3532519 ๐ [PP-1] Implement clear chat history feature in XB AI Active .
- ๐ฎ๐ณIndia narendraR Jaipur, India
Now, the chat history is also cleared when the AI panel is closed and opened again.
- ๐ฎ๐ณIndia utkarsh_33
I tested this and i can confirm that the message chat history is cleared when the AI panel is closed.I noticed an issue while testing this:-
Processing state on 0.x:-
Processing state on this MR:-
The processing states are not correct. - ๐ฎ๐ณIndia narendraR Jaipur, India
Thanks @utkarsh_33 for testing this issue. This is fixed now.
- ๐ฎ๐ณIndia utkarsh_33
When a user refreshes the page, the panelโs state (open or closed) and its content should be preserved. If the panel was open before the refresh, it should remain open with its content intact after the refresh. The history should only be cleared when the user manually closes the panel, not upon a page refresh.
- ๐ฎ๐ณIndia narendraR Jaipur, India
This is an existing issue and is not related to the current implementation. I have created a follow-up issue to address this. ๐ AI panel should preserve its state on refresh Active
- ๐ฎ๐ณIndia utkarsh_33
Since the follow-up is created and the current functionality reported in the issue is also working fine and the code also looks good to me, marking it RTBC.
- First commit to issue fork.
-
balintbrews โ
committed 800fb48f on 0.x authored by
narendrar โ
Issue #3531955 by narendrar, utkarsh_33: AI-created new components cause...
-
balintbrews โ
committed 800fb48f on 0.x authored by
narendrar โ