Skip to content

[8.x] [Inference Connector] Changed UI/UX due to the new RFC for the _inference/_service (#203363)#204690

Merged
pgayvallet merged 1 commit intoelastic:8.xfrom
pgayvallet:backport/8.x/pr-203363
Dec 18, 2024
Merged

[8.x] [Inference Connector] Changed UI/UX due to the new RFC for the _inference/_service (#203363)#204690
pgayvallet merged 1 commit intoelastic:8.xfrom
pgayvallet:backport/8.x/pr-203363

Conversation

@pgayvallet
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…ence/_service (elastic#203363)

Related RFC
https://docs.google.com/document/d/1DbWpqEKM-MJR2cSJNKLC7RcCNXXQ-_iUShsXKRFKXfk/edit?tab=t.0

## Summary

- removed Task Settings from the UI and schema definition, due to the
discussion on Inference sync Dec 5th.
- renamed provider to service
- added name and description, use name for the service selector user
friendly way
- dropped options and display type dropdown select, use freeform text
input instead
- dropped `display` field type, renamed `tooltip` to the `description`.
Properly updated `ConnectorConfigurationFormItems` in
`x-pack/plugins/stack_connectors/public/connector_types/lib/dynamic_config/connector_configuration_form_items.tsx`

UI with the updates:
<img width="1281" alt="Screenshot 2024-12-08 at 10 09 52 PM"
src="https://github.com/user-attachments/assets/fdb17dd4-c8e4-496b-85e7-03c363546b8e">

---------

Co-authored-by: Ying <ying.mao@elastic.co>
(cherry picked from commit 5b10845)
@pgayvallet pgayvallet merged commit 246ef2a into elastic:8.x Dec 18, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
stackConnectors 664.6KB 659.5KB -5.2KB
Unknown metric groups

ESLint disabled line counts

id before after diff
stackConnectors 129 128 -1

Total ESLint disabled count

id before after diff
stackConnectors 135 134 -1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants