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 #136

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

Serafin-dev
Copy link

@Serafin-dev Serafin-dev commented Oct 9, 2024

Related tickets

https://github.com/Pearson-Advance/pearson-core/tree/vue/PADV-1684

Original PR

#131

Description:

This PR is part of the AI assistance project and aims to add the functionality for the frontend to know wether disable or enable the feature.

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:

@Serafin-dev Serafin-dev force-pushed the vue/PADV-1684.re-opened branch 4 times, most recently from 4b1eba0 to 069f1ba Compare October 11, 2024 17:24
@Serafin-dev Serafin-dev force-pushed the vue/PADV-1684.re-opened branch from 069f1ba to ba1191f Compare October 11, 2024 17:26
@Serafin-dev Serafin-dev merged commit cdba011 into pearson-release/olive.stage Oct 15, 2024
13 of 35 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.

2 participants