Skip to content
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

UHF-8005: Add Talbotti (Watson) to ChatLeijuke #434

Merged
merged 1 commit into from
Feb 14, 2023

Conversation

juho-lehmonen
Copy link
Contributor

@juho-lehmonen juho-lehmonen commented Feb 8, 2023

UHF-8005

What was done

Add Talbotti / Watson chabot implemented by IBM to ChatLeijuke libraries and block form.

How to install

  • Test on any instance you have working currently
  • Make sure your instance is up and running on latest dev branch.
    • git pull origin dev
    • make fresh
  • Update the Helfi Platform config
    • composer require drupal/helfi_platform_config:dev-UHF-8005-add-watson-talbotti-to-leijuke
  • Run make drush-cr

How to test

  • Check that the new library is available in the ChatLeijuke block settings form
    • Go to block layout config: /admin/structure/block
    • Add a "Chat Leijuke" block to the "Attachments region"
      • Add whatever as the title
      • Select "Talbotti (watson)" from the Dropdown
      • Add "Talbotti" as the chat title
      • Configure block visibility so that it's visible only on finnish and on a specific page
  • Check that no external scripts are loaded if cookies aren't accepted
    • Open the page you specified
    • None of the scripts from https://coh-chat-app-prod.ow6i4n9pdzm.eu-de.codeengine.appdomain.cloud should be loaded if you haven't accepted cookies. The button might still show up though (that's how it is on the other Leijuke implementations too).
  • Check that the chatbot opens once cookies are accepted (chat cookie category)
    • Once you have accepted cookies, the button should work and pop up the chatbot. If it doesn't, try reloading the page.
    • Unaccept all cookies from /cookie-information-and-settings and make sure the chatbot is blocked again
    • Accept the chat cookie category only and check that the bot starts working
  • Check that code follows our standards

Other PRs

@sonarcloud
Copy link

sonarcloud bot commented Feb 8, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@juho-lehmonen juho-lehmonen changed the title Draft: UHF-8005: Add Talbotti (Watson) to ChatLeijuke UHF-8005: Add Talbotti (Watson) to ChatLeijuke Feb 8, 2023
@juho-lehmonen juho-lehmonen marked this pull request as ready for review February 8, 2023 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants