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

Task/244054/Recommendation Chunk Text option on QA Visualiser #966

Merged

Conversation

jag-nahl-airelogic
Copy link
Collaborator

Overview

Addresses ticket #244054

Gives an option to include the recommendation chunk text on the QA Visualiser.

Changes

Minor

  • Added new paginated function in the contentful respository
  • Added new API in the CMSController to return answers and recommendations
  • QA-Visualiser updated to use above API.
  • Added option on QA-Vis.yaml workflow so recommendations can be included/not included.

Non-functional changes (e.g. documentation)

  • Readme updated on QA-visualiser to include new environment variable

How to review the PR

  • Follow instructions in readme around setting up QA Visualiser locally
  • Set DISPLAY_RECOMMENDATIONS to true
  • Generate visualisations and confirm recommendation header text appears
  • Set DISPLAY_RECOMMENDATIONS to false
  • No recommendations header text should appear on visualisations.

Checklist

Delete any rows that do not apply to the PR.

  • PR targets development branch
  • Unit tests have been added/updated
  • GitHub workflows/actions have been added/updated
  • Documentation has been updated where relevant

@jag-nahl-airelogic jag-nahl-airelogic merged commit bee6860 into development Feb 26, 2025
12 checks passed
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.

3 participants