asadjan4611 opened a new pull request, #17989:
URL: https://github.com/apache/dolphinscheduler/pull/17989
## Purpose of the pull request
This pull request fixes a UI bug in the workflow definition list page where
the table remained in loading state after a failed list request, preventing
further retries (search, pagination, refresh) until page reload.
## Brief change log
- Update workflow definition list fetch flow to always release loading lock
on request completion.
- Ensure `loadingRef` is reset in a `finally` block for both success and
failure paths.
- Add error handling for failed requests to provide user feedback and allow
immediate retry.
- Keep the fix minimal and scoped to the workflow definition list table data
loading logic only.
## Verify this pull request
This change added no new test files and can be verified as follows:
- Open **Project -> Workflow Definition List**.
- Simulate a request failure (e.g. browser offline or blocked API request).
- Trigger list loading (initial load/search/pagination) and confirm request
fails.
- Restore network and trigger search/page change again.
- Verify:
- table is no longer locked in loading state,
- request can be retried immediately,
- data can be fetched successfully after recovery.
## Pull Request Notice
[Pull Request
Notice](https://github.com/apache/dolphinscheduler/blob/dev/docs/docs/en/contribute/join/pull-request.md)
If your pull request contains incompatible change, you should also add it to
`docs/docs/en/guide/upgrade/incompatible.md`
## fixes [17984](https://github.com/apache/dolphinscheduler/issues/17984)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
