Conversation
Coverage report
Test suite run success3643 tests passing in 810 suites. Report generated by 🧪jest coverage report action from 2e45e21 |
Coverage reportCaution Getting code coverage data failed.
Test suite run success1087 tests passing in 256 suites. Report generated by 🧪jest coverage report action from fb3bbf0 |
Coverage reportCaution Getting code coverage data failed.
Test suite run success186 tests passing in 38 suites. Report generated by 🧪jest coverage report action from 489c7d7 |
Coverage reportCaution Getting code coverage data failed.
Test suite run success348 tests passing in 74 suites. Report generated by 🧪jest coverage report action from 489c7d7 |
…r data export workflow
- DEV-022: Extend WorkFlowStepTemplateReqV1TypeEnum with export_review/export_execute;
add WorkflowTemplateTypeEnum (workflow/data_export); add workflow_type field to
IWorkflowTemplateDetailResV1
- DEV-023: Add optional workflow_type param to getWorkflowTemplateV1 and
updateWorkflowTemplateV1; add new getWorkflowTemplateListV1 function calling
GET /v1/projects/{name}/workflow_templates; add IGetWorkflowTemplateListResV1 type
- DEV-029: Add zh-CN and en-US i18n keys in sqle workflowTemplate (list table columns,
type tags, edit page titles, export review/execute step descriptions) and base
dmsDataExport (approval process preview section)
refs actiontech/dms-ee#784
(cherry picked from commit 4dd5e5c985fb49d2684609cf0530bfe86d686678)
…iew to list view - DEV-024: Replace WorkflowTemplateStepInfo/AuthInfo with ActiontechTable - Add column.tsx with workflow_type tag, approval node description, update time columns - Refactor actions.tsx to use table row edit button with permission control - Edit button navigates with workflowType query parameter - Add getWorkflowTemplateList mock API and update test files - Remove old snapshots for updated page structure refs actiontech/dms-ee#784 (cherry picked from commit d6789673acb3de193e7cfa9119b8ac4f4b905d9e)
- DEV-025: Read workflowType from URL query params, switch step type enums (export_review/export_execute vs sql_review/sql_execute) and page title based on workflow type, pass workflow_type to API calls - DEV-026: Hide AllowSubmitWhenLessAuditLevel field in BasicInfo component when editing data_export type template - Add query field to PROGRESS.update route definition for workflowType - Fix TS error in column.tsx stepTypeNameMap by using I18nKey type - Prettier formatting fixes refs actiontech/dms-ee#784 (cherry picked from commit 966319e09eb8dd6f0389c32118d9f9dbe28654c2)
…ort creation page - DEV-027: New ApprovalProcessPreview component with read-only workflow step display - Calls getWorkflowTemplateV1 with workflow_type=data_export to fetch template - Renders step list with step number, step type name, and assignee info - Shows "match by permission" when approved_by_authorized/execute_by_authorized is true - Loading state with Spin, error state with Result fallback - Bottom hint guiding users to modify approval process in Project Configure - DEV-028: Integrate preview between BasicInfoWrapper and AuditResultList in SubmitWorkflow - Add step type name i18n keys (export_review/export_execute) to base package dmsDataExport refs actiontech/dms-ee#784 (cherry picked from commit 58397af27ee9914de241e531e8c146e3d613bdcc)
…components - DEV-030: Update WorkflowTemplateDetail list page tests to cover edit button navigation with workflowType param, fix CE test missing usePermission mock - DEV-031: Update UpdateWorkflowTemplate edit page tests to mock useTypedQuery for workflowType, add data_export type title and step type tests, update BasicInfo test for hidden audit level field - DEV-032: Add ApprovalProcessPreview component tests covering loading state, success rendering, error fallback, step type name mapping, assignee display, and empty projectName guard refs actiontech/dms-ee#784 (cherry picked from commit 9b4a1b1ac6d301c2f4cc182503a934877c025d03)
…fix prettier formatting 为 dataExportWorkflowTemplateData 添加 IWorkflowTemplateDetailResV1 类型注解， 解决 assignee_user_id_list 空数组被推断为 never[] 导致的 TS2322 编译错误； 同时移除不合法的空字符串 allow_submit_when_less_audit_level 枚举值； 修复两个测试文件的 prettier 格式问题。 (cherry picked from commit 5b8b7238c1b9df63e1d3a6bb35a1b55b7fd5157d)
- Issue 1: Change workflow template page from list to tab-based layout with separate tabs for SQL exec workflow and data export - Issue 2: Fix i18n text in template edit page for data export context (e.g. "执行上线" -> "执行导出", match permission text) - Issue 3: Fix description truncation in export order detail page by replacing BasicTypographyEllipsis with plain div - Issue 4: Restore polling logic for export status refresh in detail page - Issue 5: Fix workflow progress display to show dynamic approval nodes instead of hardcoded 3 steps - Issue 6: Fix approval flow i18n mixing in create export review page (Chinese text was mixed with English labels) (cherry picked from commit d5b3e259538d63bebb99871e945d3b9d5259a76c)
…approval preview 1. Replace antd Tabs with SegmentedTabs for workflow template detail page to match audit rule template page styling convention. 2. Fix redirect link after saving data export workflow template: the "view updated template" button now navigates to the data export tab instead of always landing on the workflow tab. 3. Improve approval process preview styling in data export creation page with step indicator dots, connector lines, and better visual hierarchy. Refs: actiontech/dms-ee#784 (cherry picked from commit 56f11e29002f7c29427b4d342647bb60a1276aa8)
…review 1. Prevent removing last audit node (min 1) and hide close button when only 1 review node exists; max 4 already enforced by disabled button. 2. Data export exec step shows 'workflow creator' as executor (read-only, not modifiable) in template editor and step card display. 3. Remove ApprovalProcessPreview from SQL audit page (SubmitWorkflow). (cherry picked from commit 756a2dcfb9f7166784c9a2d226ce189dd1c12d78)
…port template Align test mock data with the updated default data export workflow template which now includes both export_review and export_execute steps. (cherry picked from commit 009db1a54953e8022008e2f4e7a18c49a832eadd)
…_execute step Update ApprovalProcessPreview test to handle multiple "按权限匹配" text elements since the default template now has both export_review and export_execute steps. Update related snapshots. (cherry picked from commit 2d249f465f59266e88730815a35167683df95b66)
Avoid ArtiomTr/jest-coverage-report-action running npm install and npx jest for base coverage collection, which fails in pnpm workspace projects. Rely on the pre-generated per-package coverage report.json instead. Made-with: Cursor
Coverage report
Test suite run success186 tests passing in 38 suites. Report generated by 🧪jest coverage report action from 1f5ad7a |
Coverage report
Test suite run success348 tests passing in 74 suites. Report generated by 🧪jest coverage report action from 1f5ad7a |
Coverage report
Test suite run success1093 tests passing in 257 suites. Report generated by 🧪jest coverage report action from 1f5ad7a |
Coverage report (@actiontech/shared)
Test suite run success186 tests passing in 38 suites. Report generated by 🧪jest coverage report action from 88def2d |
Coverage report (@actiontech/dms-kit)
Test suite run success348 tests passing in 74 suites. Report generated by 🧪jest coverage report action from 88def2d |
Coverage report (base)
Test suite run success1093 tests passing in 257 suites. Report generated by 🧪jest coverage report action from 88def2d |
Coverage report (sqle)
Test suite run success2024 tests passing in 442 suites. Report generated by 🧪jest coverage report action from 88def2d |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
不急着合并，目前只做了简单的开发环境和生产环境验证。
记录文档：http://10.186.18.11/confluence/pages/viewpage.action?pageId=246219688