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

[Streams] Schema Editor refresh button #208093

Merged
merged 7 commits into from
Jan 31, 2025

Conversation

Kerry350
Copy link
Contributor

@Kerry350 Kerry350 commented Jan 23, 2025

Summary

Implements: #207066 (comment)

I'm not sure how "smart" we need this to be. The <EuiProgress /> component already shows when either of these requests are running, but the table doesn't auto-scroll back to the top when items are reloaded (for example).

@Kerry350 Kerry350 added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 backport:version Backport to applied version labels v8.18.0 Feature:Streams This is the label for the Streams Project labels Jan 23, 2025
@Kerry350 Kerry350 self-assigned this Jan 23, 2025
@Kerry350 Kerry350 requested a review from a team as a code owner January 23, 2025 16:10
Copy link
Contributor

@flash1293 flash1293 left a comment

Choose a reason for hiding this comment

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

Works as expected, LGTM

@elasticmachine
Copy link
Contributor

elasticmachine commented Jan 31, 2025

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #63 / EQL execution logic API @ess @serverless @serverlessQA EQL type rules parses shard failures for EQL event query
  • [job] [logs] FTR Configs #63 / EQL execution logic API @ess @serverless @serverlessQA EQL type rules parses shard failures for EQL event query

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
streamsApp 283.8KB 284.1KB +297.0B

History

cc @Kerry350

@Kerry350 Kerry350 merged commit 9936dee into elastic:main Jan 31, 2025
13 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.18, 8.x, 9.0

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

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

Implements:
elastic#207066 (comment)

I'm not sure how "smart" we need this to be. The `<EuiProgress />`
component already shows when either of these requests are running, but
the table doesn't auto-scroll back to the top when items are reloaded
(for example).

---------

Co-authored-by: Joe Reuter <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 9936dee)
@kibanamachine
Copy link
Contributor

💔 Some backports could not be created

Status Branch Result
8.18 Backport failed because of merge conflicts
8.x
9.0 Backport failed because of merge conflicts

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 208093

Questions ?

Please refer to the Backport tool documentation

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

This will backport the following commits from `main` to `8.x`:
- [[Streams] Schema Editor refresh button
(#208093)](#208093)

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

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

<!--BACKPORT [{"author":{"name":"Kerry
Gallagher","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-31T12:32:48Z","message":"[Streams]
Schema Editor refresh button (#208093)\n\n##
Summary\r\n\r\nImplements:\r\nhttps://github.com//pull/207066#issuecomment-2609651683\r\n\r\nI'm
not sure how \"smart\" we need this to be. The `<EuiProgress
/>`\r\ncomponent already shows when either of these requests are
running, but\r\nthe table doesn't auto-scroll back to the top when items
are reloaded\r\n(for example).\r\n\r\n---------\r\n\r\nCo-authored-by:
Joe Reuter <[email protected]>\r\nCo-authored-by: kibanamachine
<[email protected]>","sha":"9936dee95d5c1dddfd496e3307847782523bb15f","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:version","v8.18.0","Feature:Streams","v9.1.0","v8.19.0"],"title":"[Streams]
Schema Editor refresh
button","number":208093,"url":"https://github.com/elastic/kibana/pull/208093","mergeCommit":{"message":"[Streams]
Schema Editor refresh button (#208093)\n\n##
Summary\r\n\r\nImplements:\r\nhttps://github.com//pull/207066#issuecomment-2609651683\r\n\r\nI'm
not sure how \"smart\" we need this to be. The `<EuiProgress
/>`\r\ncomponent already shows when either of these requests are
running, but\r\nthe table doesn't auto-scroll back to the top when items
are reloaded\r\n(for example).\r\n\r\n---------\r\n\r\nCo-authored-by:
Joe Reuter <[email protected]>\r\nCo-authored-by: kibanamachine
<[email protected]>","sha":"9936dee95d5c1dddfd496e3307847782523bb15f"}},"sourceBranch":"main","suggestedTargetBranches":["9.0","8.18","8.x"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/208093","number":208093,"mergeCommit":{"message":"[Streams]
Schema Editor refresh button (#208093)\n\n##
Summary\r\n\r\nImplements:\r\nhttps://github.com//pull/207066#issuecomment-2609651683\r\n\r\nI'm
not sure how \"smart\" we need this to be. The `<EuiProgress
/>`\r\ncomponent already shows when either of these requests are
running, but\r\nthe table doesn't auto-scroll back to the top when items
are reloaded\r\n(for example).\r\n\r\n---------\r\n\r\nCo-authored-by:
Joe Reuter <[email protected]>\r\nCo-authored-by: kibanamachine
<[email protected]>","sha":"9936dee95d5c1dddfd496e3307847782523bb15f"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Kerry Gallagher <[email protected]>
@tonyghiani tonyghiani removed the v8.18.0 label Feb 5, 2025
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 Feature:Streams This is the label for the Streams Project release_note:skip Skip the PR/issue when compiling release notes v8.19.0 v9.0.0 v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants