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

PADV 1684 - Add feature flag for LLM summarize for frontend #131

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

alexjmpb
Copy link

Description:

This PR is part of the AI assistance project and aims to add the connection to the summarize backend to the annotator js llm_summarize plugin. The plugin sends a request to pearson core summarize api and replaces the annotation text area value with the summary.

How to test:

  • Setup devstack and pull changes.
  • Setup edx notes api service.
  • Setup Pearson core plugin
  • Follow instructions to enable LLM summarize endpoint in pearson core.
  • Enable the waffle switch 'pearson_core.llm_summarize.enable_llm_summarize'
  • On studio, select a course, go to advance settings, and on the 'other course settings field' put the following JSON:
{
  "llm_summarize": {
    "is_enabled": true
  }
}
  • Select a text inside a text or HTML block.
  • Click on the summarize button.
  • A loader should appear indicating that the content is being summarized.
  • Now disable the switch or the other course settings, either way, the summarize button should not appear now.

Reviewers:

@alexjmpb alexjmpb changed the title feat: adding toggling of llm summarize feature PADV 1684 - Add feature flag for LLM summarize for frontend Sep 27, 2024
@Serafin-dev Serafin-dev force-pushed the vue/PADV-1584 branch 2 times, most recently from 5ffa18c to 5845715 Compare October 8, 2024 11:50
@Serafin-dev Serafin-dev merged commit b264959 into vue/PADV-1584 Oct 8, 2024
13 of 34 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