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

[8.12] Tweaks to prompt editor (#174030) #175669

Closed
wants to merge 1 commit into from

Conversation

sorenlouv
Copy link
Member

Backport

This will backport the following commits from main to 8.12:

Questions ?

Please refer to the Backport tool documentation

## Summary

This fixes the following edge case when using the chat interface:

* When the main editor has a value in the text area, and the user edits
an existing message, and presses `<enter>` on the keyboard, the value
that was in the main editor is appended as a new message, instead of
editing the existing message.

It also does a bit of cleanup (moving of ChatPromptEditor components to
`/components/prompt_editor`, and renaming to `PromptEditor`.)

### Additional fixes
* [Don't stick to bottom when changing to edit mode, re-stick to bottom
when done
editing](elastic@e8a01c1)

* [Autofocus function popover list search box upon
opening](elastic@2329d1c)

* [Remove focus trap as it wasn't doing anything
anymore](elastic@7fcb4e0)

* [Move constants used when creating monaco model inside function scope
to avoid sharing of model between multiple editor
instances](elastic@c9cab2c)

* [Disable submitting function editor when json is not
valid](elastic@2a6f1e1)

(cherry picked from commit f4265ca)

# Conflicts:
#	x-pack/plugins/observability_ai_assistant/public/components/prompt_editor/prompt_editor_function.tsx
@sorenlouv sorenlouv enabled auto-merge (squash) January 26, 2024 10:39
@apmmachine
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • /oblt-deploy-serverless : Deploy a serverless Kibana instance using the Observability test environments.
  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@sorenlouv sorenlouv closed this Jan 26, 2024
auto-merge was automatically disabled January 26, 2024 10:42

Pull request was closed

@kibana-ci
Copy link
Collaborator

kibana-ci commented Jan 26, 2024

💔 Build Failed

Failed CI Steps

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@sorenlouv sorenlouv deleted the backport/8.12/pr-174030 branch April 12, 2024 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants