Skip to content

[Streams 🌊 ] Add loader when datasources are being loaded#225005

Merged
rStelmach merged 3 commits intoelastic:mainfrom
rStelmach:fix-data-preview-table-loader
Jun 24, 2025
Merged

[Streams 🌊 ] Add loader when datasources are being loaded#225005
rStelmach merged 3 commits intoelastic:mainfrom
rStelmach:fix-data-preview-table-loader

Conversation

@rStelmach
Copy link
Contributor

@rStelmach rStelmach commented Jun 24, 2025

Summary

Current behavior
• On first load, the page immediately shows the “No samples” warning, even though the initial sample fetch is still in progress.

What this PR does
• Adds a loading indicator while data sources are being fetched, so users can see that samples are still loading instead of assuming none exist.

Demo

Before:

before.mov

After:

example.mov
@rStelmach rStelmach changed the title Add loader when datasources are being loaded Jun 24, 2025
Copy link
Contributor

@tonyghiani tonyghiani left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the update!

@rStelmach rStelmach marked this pull request as ready for review June 24, 2025 08:05
@rStelmach rStelmach requested a review from a team as a code owner June 24, 2025 08:05
@rStelmach rStelmach added backport:version Backport to applied version labels v9.1.0 v8.19.0 Feature:Streams This is the label for the Streams Project Team:obs-onboarding Observability Onboarding Team labels Jun 24, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-logs-team (Team:obs-ux-logs)

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #49 / Stateful Observability - Deployment-agnostic API integration tests SyntheticsAPITests EditMonitorsPublicAPI - Public location "before all" hook for "adds test monitor"
  • [job] [logs] FTR Configs #44 / Stateful Observability feature flag testing - Deployment-agnostic API integration tests SyntheticsCustomStatusRule "after all" hook in "SyntheticsCustomStatusRule"

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 553.9KB 554.3KB +338.0B

History

@rStelmach rStelmach merged commit 3ec2c7e into elastic:main Jun 24, 2025
10 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.19

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

@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.19

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 Jun 24, 2025
…5005) (#225066)

# Backport

This will backport the following commits from `main` to `8.19`:
- [[Streams 🌊 ] Add loader when datasources are being loaded
(#225005)](#225005)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT [{"author":{"name":"Robert
Stelmach","email":"60304951+rStelmach@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-06-24T11:59:34Z","message":"[Streams
🌊 ] Add loader when datasources are being loaded
(#225005)","sha":"3ec2c7e46e31e82a437ecd71e84525b5934e3b28","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:enhancement","Team:obs-ux-logs","backport:version","Feature:Streams","v9.1.0","v8.19.0"],"title":"[Streams
🌊 ] Add loader when datasources are being
loaded","number":225005,"url":"https://github.com/elastic/kibana/pull/225005","mergeCommit":{"message":"[Streams
🌊 ] Add loader when datasources are being loaded
(#225005)","sha":"3ec2c7e46e31e82a437ecd71e84525b5934e3b28"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/225005","number":225005,"mergeCommit":{"message":"[Streams
🌊 ] Add loader when datasources are being loaded
(#225005)","sha":"3ec2c7e46e31e82a437ecd71e84525b5934e3b28"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Robert Stelmach <60304951+rStelmach@users.noreply.github.com>
akowalska622 pushed a commit to akowalska622/kibana that referenced this pull request Jun 25, 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:enhancement Team:obs-onboarding Observability Onboarding Team v8.19.0 v9.1.0

4 participants