Skip to content

Conversation

@cactuer
Copy link

@cactuer cactuer commented May 29, 2025

image
增加时间事件的配置功能

@YunaiV YunaiV requested a review from Copilot June 15, 2025 07:51
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a configuration panel for time events to the BPMN process designer. The changes include a new TimeEventConfig panel with support for time, duration, and cycle events; dedicated components for duration (DurationConfig.vue) and cycle (CycleConfig.vue) configuration; and an update to the PropertiesPanel to integrate the new time event configuration.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
src/components/bpmnProcessDesigner/package/penal/time-event-config/TimeEventConfig.vue New panel for configuring time events with input validation and UI dialogs for time, duration, and cycle types
src/components/bpmnProcessDesigner/package/penal/time-event-config/DurationConfig.vue New component providing ISO 8601 duration configuration via presets and custom inputs
src/components/bpmnProcessDesigner/package/penal/time-event-config/CycleConfig.vue New component offering configuration for Cron expressions and standard ISO cycle formats
src/components/bpmnProcessDesigner/package/penal/PropertiesPanel.vue Updated to include the new TimeEventConfig component in the properties panel for IntermediateCatchEvent elements
Comments suppressed due to low confidence (2)

src/components/bpmnProcessDesigner/package/penal/PropertiesPanel.vue:294

  • [nitpick] Consider removing or refactoring this console.log statement before production to avoid unintentional logging in the final build.
console.log('当前eventDefinitions:', element.businessObject.eventDefinitions)

src/components/bpmnProcessDesigner/package/penal/time-event-config/TimeEventConfig.vue:260

  • The function 'toRaw' is used here without being imported from Vue. Please add 'import { toRaw } from "vue"' at the top of the file to ensure proper functionality.
bpmnInstances().modeling.updateProperties(toRaw(element), {

@YunaiV YunaiV merged commit 5a8d208 into yudaocode:master Jun 15, 2025
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.

2 participants