Skip to content

[Search] Fix mapping tab breaking docLinks and refresh#181729

Merged
sphilipse merged 2 commits intoelastic:mainfrom
sphilipse:search-fix-mapping-tab
Apr 25, 2024
Merged

[Search] Fix mapping tab breaking docLinks and refresh#181729
sphilipse merged 2 commits intoelastic:mainfrom
sphilipse:search-fix-mapping-tab

Conversation

@sphilipse
Copy link
Copy Markdown
Member

@sphilipse sphilipse commented Apr 25, 2024

Summary

This fixes two issues on the mappings tab in Search:

  • A frequent refresh caused by input changes unrelated to the mappings component
  • Doclinks breaking because they hadn't been initialized yet

@sphilipse sphilipse added release_note:skip Skip the PR/issue when compiling release notes Team:EnterpriseSearch v8.14.0 labels Apr 25, 2024
@sphilipse sphilipse requested a review from a team as a code owner April 25, 2024 15:10
@sphilipse sphilipse requested a review from a team April 25, 2024 15:10
<EuiSpacer size="s" />
<EuiLink href={docLinks.connectorsMappings} target="_blank" external>
<EuiLink
data-test-subj="enterpriseSearchSearchIndexIndexMappingsLearnHowToCustomizeIndexMappingsAndSettingsLink"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nitpick: wrong schema for test/telemetry. We should update the autofix for the plugin.

@sphilipse sphilipse enabled auto-merge (squash) April 25, 2024 15:33
@kibana-ci
Copy link
Copy Markdown

💚 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
enterpriseSearch 2.7MB 2.7MB +212.0B
indexManagement 645.3KB 645.3KB +1.0B
total +213.0B

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

@sphilipse sphilipse merged commit 35974ca into elastic:main Apr 25, 2024
@kibanamachine
Copy link
Copy Markdown
Contributor

💔 All backports failed

Status Branch Result
8.14 Backport failed because of merge conflicts

You might need to backport the following PRs to 8.14:
- [a11y] Remove jext-axe and supporting code from kibana (#180694)
- [ML] Removing datafeed preview frozen tier message in serverless (#181440)
- [ML] Hide file upload doc count chart until data is searchable (#181460)
- [Security] Quality Gate multi Organization for projects (#181027)
- [Security Solution] action not allowed (405) is shown for Duplicating Shared Exception Lists (#177814) (#178674)

Manual backport

To create the backport manually run:

node scripts/backport --pr 181729

Questions ?

Please refer to the Backport tool documentation

@sphilipse
Copy link
Copy Markdown
Member Author

💚 All backports created successfully

Status Branch Result
8.14

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

sphilipse added a commit to sphilipse/kibana that referenced this pull request Apr 26, 2024
## Summary

This fixes two issues on the mappings tab in Search:
- A frequent refresh caused by input changes unrelated to the mappings
component
- Doclinks breaking because they hadn't been initialized yet

(cherry picked from commit 35974ca)

# Conflicts:
#	x-pack/plugins/index_management/public/application/components/mappings_editor/components/document_fields/field_parameters/select_inference_id.tsx
sphilipse added a commit that referenced this pull request Apr 26, 2024
… (#181806)

# Backport

This will backport the following commits from `main` to `8.14`:
- [[Search] Fix mapping tab breaking docLinks and refresh
(#181729)](#181729)

<!--- Backport version: 8.9.8 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Sander
Philipse","email":"94373878+sphilipse@users.noreply.github.com"},"sourceCommit":{"committedDate":"2024-04-25T17:44:49Z","message":"[Search]
Fix mapping tab breaking docLinks and refresh (#181729)\n\n##
Summary\r\n\r\nThis fixes two issues on the mappings tab in Search:\r\n-
A frequent refresh caused by input changes unrelated to the
mappings\r\ncomponent\r\n- Doclinks breaking because they hadn't been
initialized
yet","sha":"35974ca8166ebc702308f04b9333c47b9ff04cfe","branchLabelMapping":{"^v8.15.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:EnterpriseSearch","v8.14.0","v8.15.0"],"number":181729,"url":"https://github.com/elastic/kibana/pull/181729","mergeCommit":{"message":"[Search]
Fix mapping tab breaking docLinks and refresh (#181729)\n\n##
Summary\r\n\r\nThis fixes two issues on the mappings tab in Search:\r\n-
A frequent refresh caused by input changes unrelated to the
mappings\r\ncomponent\r\n- Doclinks breaking because they hadn't been
initialized
yet","sha":"35974ca8166ebc702308f04b9333c47b9ff04cfe"}},"sourceBranch":"main","suggestedTargetBranches":["8.14"],"targetPullRequestStates":[{"branch":"8.14","label":"v8.14.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.15.0","labelRegex":"^v8.15.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/181729","number":181729,"mergeCommit":{"message":"[Search]
Fix mapping tab breaking docLinks and refresh (#181729)\n\n##
Summary\r\n\r\nThis fixes two issues on the mappings tab in Search:\r\n-
A frequent refresh caused by input changes unrelated to the
mappings\r\ncomponent\r\n- Doclinks breaking because they hadn't been
initialized yet","sha":"35974ca8166ebc702308f04b9333c47b9ff04cfe"}}]}]
BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:EnterpriseSearch v8.14.0 v8.15.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants