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

[v13] [Assist] New UI & rewrite #27791

Merged
merged 2 commits into from Jun 15, 2023
Merged

Conversation

ryanclark
Copy link
Contributor

Backport #27629 to branch/v13.

* Rewrite Assist & implement a new UI

* Use the node name from the API

* Code review comments

* Address UI code review comments

* Add some tests for the service

* Rename "MiniAssit" to "Assist"

* Impose a max height for the floating assist for small windows

* Add missing license header

* Update web/packages/teleport/src/Assist/Conversation/ExecuteRemoteCommandEntry.tsx

Co-authored-by: Lisa Kim <lisa@goteleport.com>

* Code review improvements

* Only use stopPropagation so links still work and assist does not close

* Make errors only show when error message isn't null

* Add websocket refresh to avoid session expirations

* Mark the conversation as not streaming when assist returns a full message

* Improvements to error handling and icon spacing

* Use encodeChallengeResponse to match the MFA backend changes

* Run prettier

* Usability improvements

---------

Co-authored-by: Lisa Kim <lisa@goteleport.com>
@public-teleport-github-review-bot

@ryanclark - this PR will require admin approval to merge due to its size. Consider breaking it up into a series smaller changes.

@ryanclark ryanclark added this pull request to the merge queue Jun 15, 2023
Merged via the queue into branch/v13 with commit 0154aff Jun 15, 2023
24 checks passed
@ryanclark ryanclark deleted the ryan/v13-backport-assist-ui branch June 15, 2023 16:35
@r0mant r0mant mentioned this pull request Jul 14, 2023
@fheinecke fheinecke mentioned this pull request Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants