mirror of
https://github.com/elastic/kibana.git
synced 2025-04-24 01:38:56 -04:00
[Inference] Updating link text for model_id (#211087)
## Summary Updating link text for `model_field` field to have `Learn more.` when the provider is Elasticsearch. ### screenshots: #### Elasticsearch provider  #### Other providers  ### Checklist Check the PR satisfies following conditions. Reviewers should verify this PR satisfies this list as well. - [X] Any text added follows [EUI's writing guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md) - [X] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios
This commit is contained in:
parent
11211cfb27
commit
84fdbcba62
3 changed files with 5 additions and 5 deletions
|
@ -37,7 +37,7 @@ describe('ConfigurationFormItems', () => {
|
|||
it('renders link when isInternalProvider is true and key is model_id', () => {
|
||||
render(<ConfigurationFormItems {...defaultProps} isInternalProvider={true} />);
|
||||
|
||||
const link = screen.getByRole('link', { name: /looking for elasticsearch model ids/i });
|
||||
const link = screen.getByRole('link', { name: /Learn more./i });
|
||||
expect(link).toBeInTheDocument();
|
||||
expect(link).toHaveAttribute(
|
||||
'href',
|
||||
|
|
|
@ -66,7 +66,7 @@ export const ConfigurationFormItems: React.FC<ConfigurationFormItemsProps> = ({
|
|||
external
|
||||
target="_blank"
|
||||
>
|
||||
{LABELS.ES_MODELS_LINK_TEXT}
|
||||
{LABELS.LEARN_MORE}
|
||||
</EuiLink>
|
||||
</>
|
||||
) : (
|
||||
|
|
|
@ -120,10 +120,10 @@ export const OPTIONALTEXT = i18n.translate(
|
|||
}
|
||||
);
|
||||
|
||||
export const ES_MODELS_LINK_TEXT = i18n.translate(
|
||||
'xpack.inferenceEndpointUICommon.components.esModelsLinkText',
|
||||
export const LEARN_MORE = i18n.translate(
|
||||
'xpack.inferenceEndpointUICommon.components.learnMoreText',
|
||||
{
|
||||
defaultMessage: 'Looking for Elasticsearch model Ids',
|
||||
defaultMessage: 'Learn more.',
|
||||
}
|
||||
);
|
||||
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue