NCIOCPL / cgov-digital-platform

The Cancer.gov Digital Communications Platform
GNU General Public License v2.0
11 stars 33 forks source link

CTS Proactive Chat pop-up update #3444

Closed santwuma closed 2 years ago

santwuma commented 2 years ago

We need to update the proactive chat pop-up url (due to Livehelp upgrade) so the user experience for Livehelp on CGov will be consistent. Related JIRA ticket https://tracker.nci.nih.gov/browse/CGOV-16414

ESTIMATE low effort, code change due by 7/30/22

Acceptance Criteria

Scenario: Proactive pop-up "Chat Now" link
  Given i'm a user on cancer.gov
    And browsing through the Clinical Trial pages
  When  the proactive chat pop up and i click on the "Chat Now" link
  Then  i should be directed to livehelpcancer.gov launch window (url = https://livehelp.cancer.gov/app/chat/chat_launch)

Resources:

Notes

Approach approved by ODDC leads during 7/15 meeting

Solution

Prerequisites

Tasks

Technical Notes

venglisch commented 2 years ago

Does a page in the DP container exist that lets me test the Spanish version of the proactive chat pop-up? Who would know?

venglisch commented 2 years ago

Note regarding the link. The link will point to

The user is automatically directed to "/app/chat/chat_launch"

kate-mashkina commented 2 years ago

Test URLs @alinai @SupriyaSaxena0605 :

Live Chat on the home page cancer.gov is your reference, it is the page the cts chat should take you the path should be /app/chat/chat_launch and the base url will be different from production ( e.g. this https://nci--tst.custhelp.com/app/chat/chat_launch is expected url on ODE for english ) Spanish: https://ncigovcdode491.prod.acquia-sites.com/espanol/cancer/tratamiento/medicamentos English: https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/search https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/search/advanced https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/search/r?loc=0&rl=2 https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/search/v?id=NCI-2014-01507&loc=0&rl=2 https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/disease/breast-cancer https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/intervention/trastuzumab https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/intervention/spiroplatin https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/disease/breast-cancer/treatment/spiroplatin

alinai commented 2 years ago

Test URLs @alinai @SupriyaSaxena0605 :

Live Chat on the home page cancer.gov is your reference, it is the page the cts chat should take you the path should be /app/chat/chat_launch and the base url will be different from production ( e.g. this https://nci--tst.custhelp.com/app/chat/chat_launch is expected url on ODE for english ) Spanish: https://ncigovcdode491.prod.acquia-sites.com/espanol/cancer/tratamiento/medicamentos English: https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/search https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/search/advanced https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/search/r?loc=0&rl=2 https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/search/v?id=NCI-2014-01507&loc=0&rl=2 https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/disease/breast-cancer https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/intervention/trastuzumab https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/intervention/spiroplatin https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/disease/breast-cancer/treatment/spiroplatin

Thanks @kate-mashkina for clarifying the requirement!

alinai commented 2 years ago

@santwuma , @kate-mashkina , "Status" is missing on ODE in both English and Spanish:

image image

alinai commented 2 years ago

The font size is not same for the text "Chat support is available". Also the text marked in the screen-shot has smaller fonts on ODE. image

venglisch commented 2 years ago

I thought we're just testing the proactive chat window?

venglisch commented 2 years ago

Yes, the "Status" label has been dropped from the chat window.

I also adjusted the font sizes because some sections used a larger font than others.

(I can't sneak anything passed your eyes, can I? )

alinai commented 2 years ago

Yes, the "Status" label has been dropped from the chat window.

I also adjusted the font sizes because some sections used a larger font than others.

(I can't sneak anything passed your eyes, can I? )

Thanks for confirming @venglisch.

alinai commented 2 years ago

Noticed Prod issue in the English version: The Spanish url on the English version has "L" and "H" in upper case in the url in both Prod and ODE. On clicking "Iniciar Chat" we are redirected to the url https://livehelp-es.cancer.gov/app/chat/chat_launch. Not sure why we show https://LiveHelp-es.cancer.gov/ on the page. image

alinai commented 2 years ago

@kate-mashkina, @santwuma , verified that English: Clicking "Chat Now" button on Basic CTS Form user is redirected to : https://nci--tst.custhelp.com/app/chat/chat_launch Clicking "Chat Now" button on CTS Search Results page user is redirected to :: https://nci--tst.custhelp.com/app/chat/chat_launch Clicking "Chat Now" button on Advanced CTS Form user is redirected to : https://nci--tst.custhelp.com/app/chat/chat_launch Clicking "Chat Now" button on CTS View page user is redirected to : https://nci--tst.custhelp.com/app/chat/chat_launch Clicking "Chat Now" button on No trial page (https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/search/r?loc=0&q=penguin&rl=1) user is redirected to : redirected to https://nci--tst.custhelp.com/app/chat/chat_launch Clicking "Chat Now" button on Invalid Param page (https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/search/r?a=190&loc=0&rl=1) user is redirected to : https://nci--tst.custhelp.com/app/chat/chat_launch

Spanish: Clicking "Inciar Chat" button on Spanish page (https://ncigovcdode491.prod.acquia-sites.com/espanol/cancer/tratamiento/medicamentos) user is redirected to : https://nci2--tst.custhelp.com/app/chat/chat_launch

SupriyaSaxena0605 commented 2 years ago

Purple Boxes for Clinical Trials, Cancer Information and Quitting Smoking are bigger in width on ODE(200 px) as compared to Prod(180 px). image

kate-mashkina commented 2 years ago

@welshja the click event on LiveHelp chat popup's button is gone from CTS pages

alinai commented 2 years ago

@santwuma , @kate-mashkina , I don't see the hover color changing on hovering the purple buttons on ODE.

santwuma commented 2 years ago

@alinai confirming "status" and font changes are fine and previously passed UX review in JIRA (added related JIRA ticket to this ticket details). I think the https://livehelp-es.cancer.gov/ url on the pop-up box should be fine, since that's what it currently is in prod and was provided by CIS. @venglisch can confirm.

Test results look good to me. @sarinapadilla can you confirm as well?

alinai commented 2 years ago

@santwuma , @kate-mashkina , I don't see the hover color changing on hovering the purple buttons on ODE.

@santwuma , can you please confirm if the issue I reported above looks good to you as well?

SupriyaSaxena0605 commented 2 years ago

Purple Boxes for Clinical Trials, Cancer Information and Quitting Smoking are bigger in width on ODE(200 px) as compared to Prod(180 px). image

@santwuma , please confirm if this issue looks good too.

SupriyaSaxena0605 commented 2 years ago

@kate-mashkina, @santwuma , verified that English: Clicking "Chat Now" button on DLP(https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/disease/breast-cancer), user is redirected to : https://nci--tst.custhelp.com/app/chat/chat_launch Clicking "Chat Now" button on DLP(https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/intervention/trastuzumab), user is redirected to :: https://nci--tst.custhelp.com/app/chat/chat_launch Clicking "Chat Now" button on No trial disease DLP(https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/disease/breast-cancer/treatment/spiroplatin), user is redirected to : https://nci--tst.custhelp.com/app/chat/chat_launch Clicking "Chat Now" button on No trial intervention DLP(https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/intervention/spiroplatin), user is redirected to : https://nci--tst.custhelp.com/app/chat/chat_launch

venglisch commented 2 years ago

Noticed Prod issue in the English version: The Spanish url on the English version has "L" and "H" in upper case in the url in both Prod and ODE. On clicking "Iniciar Chat" we are redirected to the url https://livehelp-es.cancer.gov/app/chat/chat_launch. Not sure why we show https://LiveHelp-es.cancer.gov/ on the page. image

The hostname of a URL is not case sensitive, so this isn't an issue. Additionally, there was no change as this is how PROD displays the URL. Our logo displays the text with "L" and "H" in upper case. I assume the URL had been written this way to closely relate to that spelling.

venglisch commented 2 years ago

Purple Boxes for Clinical Trials, Cancer Information and Quitting Smoking are bigger in width on ODE(200 px) as compared to Prod(180 px). image

That's correct, my feeling was that there wasn't enough padding for the button with the "Cancer Information" label.

alinai commented 2 years ago

@santwuma , @kate-mashkina , I don't see the hover color changing on hovering the purple buttons on ODE.

@venglisch , can you please confirm if this looks good to you?

venglisch commented 2 years ago

@alinai , you may need to reload the page because I just updated the CSS within the last 30 minutes or so.

Note to @santwuma, this CSS update is happening as part of the LiveHelp framework update and is not part of the DP code.

alinai commented 2 years ago

@santwuma , On ODE491, Verified that for CTS pages (English) urls:

On ODE491, Verified that for Spanish page https://ncigovcdode491.prod.acquia-sites.com/espanol/cancer/tratamiento/medicamentos

Thanks @kate-mashkina for verifying that Analytics are working as expected. Thanks @venglisch for fixing the reported issues.

SupriyaSaxena0605 commented 2 years ago

@kate-mashkina, @santwuma , verified that English: Clicking "Chat Now" button on DLP(https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/disease/breast-cancer), user is redirected to : https://nci--tst.custhelp.com/app/chat/chat_launch Clicking "Chat Now" button on DLP(https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/intervention/trastuzumab), user is redirected to :: https://nci--tst.custhelp.com/app/chat/chat_launch Clicking "Chat Now" button on No trial disease DLP(https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/disease/breast-cancer/treatment/spiroplatin), user is redirected to : https://nci--tst.custhelp.com/app/chat/chat_launch Clicking "Chat Now" button on No trial intervention DLP(https://ncigovcdode491.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/intervention/spiroplatin), user is redirected to : https://nci--tst.custhelp.com/app/chat/chat_launch