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

[dashboard] fix Inspect panel action displays untitled instead of panel title #177516

Merged
merged 2 commits into from
Feb 22, 2024

Conversation

nreese
Copy link
Contributor

@nreese nreese commented Feb 21, 2024

Fixes #176870

PR provides a utility method for fetching title that falls back to default title when title is not provided.

@nreese nreese requested review from a team as code owners February 21, 2024 21:40
@botelastic botelastic bot added the Feature:Drilldowns Embeddable panel Drilldowns label Feb 21, 2024
@nreese nreese added Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas release_note:skip Skip the PR/issue when compiling release notes v8.13.0 project:embeddableRebuild v8.14.0 and removed Feature:Drilldowns Embeddable panel Drilldowns labels Feb 21, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

@botelastic botelastic bot added the Feature:Drilldowns Embeddable panel Drilldowns label Feb 21, 2024
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
urlDrilldown 12 34 +22

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/presentation-publishing 118 120 +2

Async chunks

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

id before after diff
dashboard 384.4KB 383.8KB -569.0B

Page load bundle

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

id before after diff
dashboardEnhanced 14.2KB 14.3KB +89.0B
presentationPanel 41.6KB 41.7KB +72.0B
urlDrilldown 15.7KB 15.8KB +101.0B
total +262.0B
Unknown metric groups

API count

id before after diff
@kbn/presentation-publishing 154 156 +2

History

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

Copy link
Contributor

@Dosant Dosant left a comment

Choose a reason for hiding this comment

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

code lgtm

Copy link
Contributor

@ThomThomson ThomThomson left a comment

Choose a reason for hiding this comment

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

Changes LGTM! Code only review.

expect(getPanelTitle(api)).toBe('default title');
});

test('should return default title when title is empty string', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Good to call this out with a test!

@nreese nreese merged commit c747a3b into elastic:main Feb 22, 2024
19 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Feb 22, 2024
…el title (elastic#177516)

Fixes elastic#176870

PR provides a utility method for fetching title that falls back to
default title when title is not provided.

(cherry picked from commit c747a3b)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.13

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 Feb 22, 2024
… of panel title (#177516) (#177610)

# Backport

This will backport the following commits from `main` to `8.13`:
- [[dashboard] fix Inspect panel action displays untitled instead of
panel title (#177516)](#177516)

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

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

<!--BACKPORT [{"author":{"name":"Nathan
Reese","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-02-22T15:27:12Z","message":"[dashboard]
fix Inspect panel action displays untitled instead of panel title
(#177516)\n\nFixes
https://github.com/elastic/kibana/issues/176870\r\n\r\nPR provides a
utility method for fetching title that falls back to\r\ndefault title
when title is not
provided.","sha":"c747a3b35dc086bfa3b8a1cbe67e276558c9a1f4","branchLabelMapping":{"^v8.14.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Presentation","release_note:skip","Feature:Drilldowns","v8.13.0","project:embeddableRebuild","v8.14.0"],"title":"[dashboard]
fix Inspect panel action displays untitled instead of panel
title","number":177516,"url":"https://github.com/elastic/kibana/pull/177516","mergeCommit":{"message":"[dashboard]
fix Inspect panel action displays untitled instead of panel title
(#177516)\n\nFixes
https://github.com/elastic/kibana/issues/176870\r\n\r\nPR provides a
utility method for fetching title that falls back to\r\ndefault title
when title is not
provided.","sha":"c747a3b35dc086bfa3b8a1cbe67e276558c9a1f4"}},"sourceBranch":"main","suggestedTargetBranches":["8.13"],"targetPullRequestStates":[{"branch":"8.13","label":"v8.13.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.14.0","branchLabelMappingKey":"^v8.14.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/177516","number":177516,"mergeCommit":{"message":"[dashboard]
fix Inspect panel action displays untitled instead of panel title
(#177516)\n\nFixes
https://github.com/elastic/kibana/issues/176870\r\n\r\nPR provides a
utility method for fetching title that falls back to\r\ndefault title
when title is not
provided.","sha":"c747a3b35dc086bfa3b8a1cbe67e276558c9a1f4"}}]}]
BACKPORT-->

Co-authored-by: Nathan Reese <[email protected]>
fkanout pushed a commit to fkanout/kibana that referenced this pull request Mar 4, 2024
…el title (elastic#177516)

Fixes elastic#176870

PR provides a utility method for fetching title that falls back to
default title when title is not provided.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Drilldowns Embeddable panel Drilldowns project:embeddableRebuild release_note:skip Skip the PR/issue when compiling release notes Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas v8.13.0 v8.14.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Dashboard] Inspect panel action displays "untitled" instead of panel title
6 participants