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

Search synonyms empty prompt #205723

Merged
merged 9 commits into from
Jan 8, 2025

Conversation

efegurkan
Copy link
Member

@efegurkan efegurkan commented Jan 7, 2025

Summary

Implement empty prompt for the Synonyms page.
image

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

Copy link
Contributor

github-actions bot commented Jan 7, 2025

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

@efegurkan efegurkan marked this pull request as ready for review January 7, 2025 14:22
@efegurkan efegurkan requested review from a team as code owners January 7, 2025 14:22
@efegurkan efegurkan added the backport:version Backport to applied version labels label Jan 7, 2025
Copy link
Contributor

@leemthompo leemthompo left a comment

Choose a reason for hiding this comment

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

  • Doclink LGTM.
  • Some copyedits for consistency, clarity and parallel sentence structures.

- Check config before registering

- Update navigation on synonyms plugins

- Fix missing config export

- Fix quick check failures

- FTR fixes

- Review changes

- [CI] Auto-commit changed files from 'node scripts/lint_ts_projects --fix'

- Fix FTRs

- Fix privilege FTR

- Rebase and review changes

Update tsconfig base path

[CI] Auto-commit changed files from 'node scripts/generate codeowners'

Fix types

Fix checks

Adds empty prompt

Get started button is not functioning yet, and implemented as a
placeholder.

Fix rebase leftovers

[CI] Auto-commit changed files from 'node scripts/lint_ts_projects --fix'
@efegurkan efegurkan force-pushed the search-synonyms-empty-prompt branch from a7d467d to 0d99ded Compare January 8, 2025 10:03
Co-authored-by: Liam Thompson <[email protected]>
@efegurkan efegurkan force-pushed the search-synonyms-empty-prompt branch from ee4e847 to 9b3e6f5 Compare January 8, 2025 10:06
Copy link
Member

@sphilipse sphilipse left a comment

Choose a reason for hiding this comment

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

LGTM!

@efegurkan efegurkan enabled auto-merge (squash) January 8, 2025 15:25
@leemthompo
Copy link
Contributor

@efegurkan could you snap an updated screenshot if possible 🙏 😄

@efegurkan
Copy link
Member Author

@leemthompo Updated the image above

@efegurkan efegurkan merged commit 5323067 into elastic:main Jan 8, 2025
9 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/12675463988

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
searchSynonyms 7 83 +76

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/doc-links 81 82 +1

Async chunks

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

id before after diff
aiAssistantManagementSelection 93.7KB 93.8KB +102.0B
lists 146.1KB 146.2KB +102.0B
searchSynonyms 790.0B 33.0KB +32.2KB
total +32.4KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
core 444.5KB 444.6KB +102.0B
searchSynonyms 3.8KB 4.3KB +572.0B
total +674.0B
Unknown metric groups

API count

id before after diff
@kbn/doc-links 81 82 +1

miscellaneous assets size

id before after diff
searchSynonyms 0.0B 161.8KB +161.8KB

History

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jan 8, 2025
## Summary

Implement empty prompt for the Synonyms page.

<img width="1044" alt="Screenshot 2025-01-07 at 13 56 09"
src="https://github.com/user-attachments/assets/d61c5251-afdb-4e34-bf18-a20ab1044800"
/>

### 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
- [ ] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

---------

Co-authored-by: Liam Thompson <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 5323067)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

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

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Jan 8, 2025
# Backport

This will backport the following commits from `main` to `8.x`:
- [Search synonyms empty prompt
(#205723)](#205723)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Efe Gürkan
YALAMAN","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-08T17:06:29Z","message":"Search
synonyms empty prompt (#205723)\n\n## Summary\r\n\r\nImplement empty
prompt for the Synonyms page.\r\n\r\n<img width=\"1044\"
alt=\"Screenshot 2025-01-07 at 13 56
09\"\r\nsrc=\"https://github.com/user-attachments/assets/d61c5251-afdb-4e34-bf18-a20ab1044800\"\r\n/>\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\r\n\r\n- [x] Any text added follows [EUI's
writing\r\nguidelines](https://elastic.github.io/eui/#/guidelines/writing),
uses\r\nsentence case text and includes
[i18n\r\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\r\n-
[x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common scenarios\r\n- [ ] [Flaky
Test\r\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1)
was\r\nused on any tests changed\r\n- [x] The PR description includes
the appropriate Release Notes section,\r\nand the correct
`release_note:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Liam Thompson
<[email protected]>\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"53230679065cd7bc00f15cdb01a3d7f592440c8f","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Search","backport:version","v8.18.0"],"title":"Search
synonyms empty
prompt","number":205723,"url":"https://github.com/elastic/kibana/pull/205723","mergeCommit":{"message":"Search
synonyms empty prompt (#205723)\n\n## Summary\r\n\r\nImplement empty
prompt for the Synonyms page.\r\n\r\n<img width=\"1044\"
alt=\"Screenshot 2025-01-07 at 13 56
09\"\r\nsrc=\"https://github.com/user-attachments/assets/d61c5251-afdb-4e34-bf18-a20ab1044800\"\r\n/>\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\r\n\r\n- [x] Any text added follows [EUI's
writing\r\nguidelines](https://elastic.github.io/eui/#/guidelines/writing),
uses\r\nsentence case text and includes
[i18n\r\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\r\n-
[x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common scenarios\r\n- [ ] [Flaky
Test\r\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1)
was\r\nused on any tests changed\r\n- [x] The PR description includes
the appropriate Release Notes section,\r\nand the correct
`release_note:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Liam Thompson
<[email protected]>\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"53230679065cd7bc00f15cdb01a3d7f592440c8f"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205723","number":205723,"mergeCommit":{"message":"Search
synonyms empty prompt (#205723)\n\n## Summary\r\n\r\nImplement empty
prompt for the Synonyms page.\r\n\r\n<img width=\"1044\"
alt=\"Screenshot 2025-01-07 at 13 56
09\"\r\nsrc=\"https://github.com/user-attachments/assets/d61c5251-afdb-4e34-bf18-a20ab1044800\"\r\n/>\r\n\r\n###
Checklist\r\n\r\nCheck the PR satisfies following conditions.
\r\n\r\nReviewers should verify this PR satisfies this list as
well.\r\n\r\n- [x] Any text added follows [EUI's
writing\r\nguidelines](https://elastic.github.io/eui/#/guidelines/writing),
uses\r\nsentence case text and includes
[i18n\r\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\r\n-
[x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common scenarios\r\n- [ ] [Flaky
Test\r\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1)
was\r\nused on any tests changed\r\n- [x] The PR description includes
the appropriate Release Notes section,\r\nand the correct
`release_note:*` label is applied per
the\r\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\r\n\r\n---------\r\n\r\nCo-authored-by:
Liam Thompson
<[email protected]>\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"53230679065cd7bc00f15cdb01a3d7f592440c8f"}},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Efe Gürkan YALAMAN <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:Search v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants