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

feat: use AI Chat js bundle component #422

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

asadali145
Copy link

@asadali145 asadali145 commented Jan 8, 2025

What are the relevant tickets?

Closes https://github.com/mitodl/hq/issues/6514

Description (What does it do?)

Uses the bundle generated through mitodl/smoot-design#32 with the Asides.

Screenshots (if appropriate):

  • Desktop screenshots
  • Mobile width screenshots
    ai-chat-bundle

How can this be tested?

  • Generate package by checking out this branch and running pants package :: .
  • Enable xBlock configurations in CMS <STUDIO_URL>/admin/xblock_config/studioconfig/
  • Enable xBlock asides in LMS <LMS_URL>/admin/lms_xblock/xblockasidesconfig/
  • Install the ol-openedx-chat in LMS, Reload them
  • Visit a course unit with MCQs and see the chat window

@asadali145 asadali145 changed the title feat: AI Chat js bundle component feat: use AI Chat js bundle component Jan 8, 2025
@asadali145 asadali145 changed the base branch from asad/basic-lms-chat-v2 to main January 15, 2025 12:12
@asadali145 asadali145 force-pushed the asad/smoot-js-bundle branch 2 times, most recently from 6836af2 to bfc9f96 Compare January 15, 2025 12:26
@asadali145 asadali145 force-pushed the asad/smoot-js-bundle branch from bfc9f96 to faeedd0 Compare January 15, 2025 12:34
@asadali145 asadali145 force-pushed the asad/smoot-js-bundle branch from 8e66c10 to 9b48764 Compare January 15, 2025 13:19
@asadali145 asadali145 force-pushed the asad/smoot-js-bundle branch from c14e729 to 6531a8a Compare January 17, 2025 12:45
@pdpinch
Copy link
Member

pdpinch commented Jan 17, 2025

will this close https://github.com/mitodl/hq/issues/6514

@asadali145
Copy link
Author

will this close mitodl/hq#6514

Yes, It will. I can add the issue in the PR description.

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