search

Yeldaai / chat-plugin

Yelda Chat Plugin
MIT License
0 stars 0 forks source link

Feat listen leave viewport #100

Closed vishnu-ravi closed 2 years ago

vishnu-ravi commented 2 years ago

i tested with naomi, and we both got the inactivity notification, but i can't get the leaveViewport one, and Naomi only succeeded with a lot of complex manipulation, can you retest ? @codebynao can you explain how you did ?

i tried both on chrome and FF

we put the build of the webchat and the injector into yelda front, that we rebuild, and we have been testing with "pommy copy" assistant, in demo mode

node_modules/yelda-webchat/dist need to be updated to test the leaveViewport notification

codebynao commented 2 years ago

i tested with naomi, and we both got the inactive notification, but i can't get the leaveViewport one, and Naomi only succeeded with a lot of complex manipulation, can you retest ?

To manage to have the leave viewport notification I followed these steps:

  1. Go to a demo page
  2. The webchat is loading in the smartphone template with the welcome message
  3. I received the inactivity notifications
  4. I moved the mouse outside the window and even switch to another app on my computer then come back: I didn't receive the leave viewport notification
  5. Still on the demo page I selected another platform (e.g. Desktop) then went back to the Mobile platform
  6. I received the leave viewport notification

I reproduced this behaviour multiple times using different browsers (Chrome, Safari, Firefox)

meelie commented 2 years ago

@vishnu-ravi in don't understand your answer :

node_modules/yelda-webchat/dist need to be updated to test the leaveViewport notification

how could we experiment the inactivity notification if the webchat injector is not already correctly set ?

vishnu-ravi commented 2 years ago

@vishnu-ravi in don't understand your answer :

node_modules/yelda-webchat/dist need to be updated to test the leaveViewport notification

how could we experiment the inactivity notification if the webchat injector is not already correctly set ?

inActivity notification triggered from webchat, but leave viewport notification triggered from chat-plugin

meelie commented 2 years ago

i still confirm that we do use the latest version of the webchat injector