Amy0104 commented on code in PR #10706:
URL: https://github.com/apache/dolphinscheduler/pull/10706#discussion_r934336070
##########
dolphinscheduler-ui/src/views/projects/workflow/definition/use-table.ts:
##########
@@ -257,6 +257,20 @@ export function useTable() {
variables.page -= 1
}
+ variables.checkedRowKeys = []
+ getTableData({
+ pageSize: variables.pageSize,
+ pageNo: variables.page,
+ searchVal: variables.searchVal
+ })
+ }).catch(() => {
+ if (
+ Math.ceil((variables.columns.length -
variables.checkedRowKeys.length) / variables.pageSize) < variables.page
+ && variables.page > 1
+ ) {
+ variables.page -= 1;
+ }
+
variables.checkedRowKeys = []
Review Comment:
Columns length means how many columns that the data table has, and it does
not seem to be related to how many workflows that the data table has. If it is
partially deleted successfully, it is hardly to know the current page number
from the front end part.
--
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]
