Skip to content

Conversation

@mkouzel-yext
Copy link
Contributor

@mkouzel-yext mkouzel-yext commented Mar 14, 2025

Updates chat-ui-react to use chat-headless-react v0.9.5, allowing us to use the latest version of yext/analytics

J=WAT-4671
TEST=auto, manual

Unit and integration tests pass.
Spun up test-site locally with sandbox model. Was able to make requests and receive responses as expected, confirmed analytics events were going through for responses, impressions, and link clicks.

Updates chat-ui-react to use chat-headless-react v0.9.5, allowing us to use the latest version of yext/analytics
@mkouzel-yext mkouzel-yext requested a review from a team as a code owner March 14, 2025 21:06
* */
const testEnvironment = 'DEV';
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
// @ts-ignore

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does it make sense to refactor it so that the urls are shared? If I'm not mistaken, the 2 URLs are the same outside of the /streaming on the 2nd one right?

@mkouzel-yext mkouzel-yext merged commit 5ccc006 into main Mar 14, 2025
9 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.

4 participants