Skip to content

debezium/dbz#1725 fix: Improve loading state-management#380

Open
gmarav05 wants to merge 1 commit into
debezium:mainfrom
gmarav05:fix/improve-loading-state-management
Open

debezium/dbz#1725 fix: Improve loading state-management#380
gmarav05 wants to merge 1 commit into
debezium:mainfrom
gmarav05:fix/improve-loading-state-management

Conversation

@gmarav05
Copy link
Copy Markdown
Contributor

Fixes debezium/dbz#1725

Description

This PR improves the initial loading state management in the Platform UI by addressing the following:

  • Removed "Loading" text from the initial application load window by updating the Suspense fallback in main.tsx.
  • Implemented retry logic: Configured the global QueryClient with retry: 3. This ensures that in case of network issues, the UI will attempt to fetch data 3 times before displaying a network failure message, rather than showing a loading state indefinitely.
  • Unified Loading UX: Removed hardcoded and translated "Loading..." text from major page components (Pipelines, Sources, Destinations, Connections, and Transforms) while retaining the PatternFly Spinner for visual feedback.
  • Updated Tests: Adjusted unit tests to verify loading states by checking for the progressbar role (associated with the Spinner component) instead of searching for the now-removed text.

PR Checklist

  • I have read the contribution guidelines and the governance document on PR expectations.
  • Minimal changes to code not directly related to your change (e.g. no unnecessary formatting changes or refactoring to existing code)
  • One feature/change per PR unless tightly coupled
  • Do a rebase on upstream main

@github-actions
Copy link
Copy Markdown

Hi @gmarav05, thanks for your contribution. Please prefix the commit message(s) with the debezium/dbz#xxx GitHub issue key.

@gmarav05 gmarav05 changed the title debezium/dbz#1725 Improve loading state-management debezium/dbz#1725 fix: Improve loading state-management Apr 29, 2026
@gmarav05 gmarav05 force-pushed the fix/improve-loading-state-management branch 2 times, most recently from fcce6ca to ee41beb Compare April 29, 2026 19:37
@gmarav05
Copy link
Copy Markdown
Contributor Author

gmarav05 commented Apr 29, 2026

Hello @mfvitale, @Naros , @indraraj

I think this is ready for the review.

Please let me know if any changes are required.

@gmarav05 gmarav05 force-pushed the fix/improve-loading-state-management branch from ee41beb to b41a6bd Compare April 29, 2026 19:42
Signed-off-by: Aravind <gmarav005@gmail.com>
@gmarav05 gmarav05 force-pushed the fix/improve-loading-state-management branch from b41a6bd to c697c48 Compare April 29, 2026 19:47
@Naros Naros requested a review from indraraj April 29, 2026 20:45
@mfvitale
Copy link
Copy Markdown
Member

@gmarav05 Are you able to record a video of the result?

@gmarav05
Copy link
Copy Markdown
Contributor Author

gmarav05 commented Apr 30, 2026

@mfvitale No, I did not record it. I will record it and post it here today. Thanks for letting me know.

Copy link
Copy Markdown
Member

@indraraj indraraj left a comment

Choose a reason for hiding this comment

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

@gmarav05 Please rebase onto the latest main. Plus these are listing pages that you covered and have pooling, retry is not needed here instead limit polling after a few failed attempts and show an appropriate message to the user.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve the initial loading state managment in the Platfrom UI

3 participants