stevqin commented on PR #18648: URL: https://github.com/apache/dolphinscheduler/pull/18648#issuecomment-5728885530
Follow-up update pushed in `6c03c89`: - Added a read-only task configuration viewer directly from each Gantt row. It uses the task definition embedded in the workflow instance DAG, so the dialog reflects the exact instance version rather than the latest workflow definition. - Kept task names selectable/copyable. Row numbers remain in a dedicated left column; the status indicator is now 7px, followed by status/type and the log/configuration actions. - Read-only task dialogs no longer apply project defaults and show only the cancel/close action. - Preserved the linear-time Gantt model construction from the previous review fix. Validation: Gantt model/layout tests (10/10), Prettier, `vue-tsc --noEmit`, and production build all pass. The same change was also smoke-tested on the 3.2.1 production UI with a 32-task workflow, including opening an SQL task configuration and verifying name, SQL, pre-SQL, datasource, retry settings, and read-only behavior. -- 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]
