Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Send event task #318

Merged
merged 61 commits into from
Aug 22, 2022
Merged

Send event task #318

merged 61 commits into from
Aug 22, 2022

Conversation

v20stepanenko
Copy link
Contributor

@v20stepanenko v20stepanenko commented Jun 16, 2022 •
edited by valb3r
Loading

Fixes #319

v20stepanenko and others added 30 commits May 25, 2022 17:53
@v20stepanenko
added send event task
153d763
@v20stepanenko
bonded simple property with Event Type and collapsed dependent proper…
906415e 
…ties in PropVisualiser
@v20stepanenko
clear dependent property when base property is empty
ebf88e0
@v20stepanenko
fix position in property visualization
7c4dd89
@v20stepanenko
fix, restore needed code
3d493ef
@v20stepanenko
added print default value by head property
69e5dda
@v20stepanenko
added feature select list property
1ecccb1
@v20stepanenko
send event task test work with xml
e9e84c3
@v20stepanenko
take out logic of adding events
4063d71
@v20stepanenko
dependent property testing
d2d12fb
@v20stepanenko
added eventInParameter property
b62ffe5
@valb3r
Merge branch 'master' into send-event-task
7ca3cfa
@v20stepanenko
added eventOutParameter property
4aafcc8
@v20stepanenko
fixed property visualiser when have more than 1 external group property
e56ceee
@v20stepanenko
fixed property visualiser when editing index property, nested prop is…
e96a700 
… hiding
@v20stepanenko
minor fix, uncommented necessary code
c4b6190
@v20stepanenko
fix update nested properties, with index
a9af61c
@v20stepanenko
fix removing nested properties, with index
bc998c0
@v20stepanenko
Merge remote-tracking branch 'origin/send-event-task' into send-event…
89a302a 
…-task
@v20stepanenko
added test to update cascade property and remove
1e7f491
@v20stepanenko
remove depend property logic basic on isHead and list dependencies
0129c41
@v20stepanenko
fix visualise property when new bpmn object created showed all props
4928b5c
@v20stepanenko
remove extra arguments
d73ec1f
@v20stepanenko
rollback removeUiOnlyProperty
96b1d2e
@v20stepanenko
add execution listener
1db4885
@v20stepanenko
add flag to create button
d4c2360
@v20stepanenko
send event task parsing xml
1d45875
@v20stepanenko
clean up
f9f6d15
@v20stepanenko
clean up
f547c0a
@v20stepanenko
Merge remote-tracking branch 'origin/send-event-task' into send-event…
789b137 
…-task
valb3r
valb3r requested changes Jul 23, 2022
View reviewed changes
Copy link
Owner

@valb3r valb3r left a comment

Choose a reason for hiding this comment

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

Newly created element looks OK:
Screenshot 2022-07-23 at 09 57 12

But already existing one is broken:
Screenshot 2022-07-23 at 09 56 59

@v20stepanenko
fix event type group property pad
0470a11
@v20stepanenko
fix event type group property pad
4fff6f5
@v20stepanenko
fix remove group property from visualiser if remove last property
2f6d264
@v20stepanenko
Merge remote-tracking branch 'origin/send-event-task' into send-event…
7df0482 
…-task

# Conflicts:
#	bpmn-intellij-plugin-core/src/main/kotlin/com/valb3r/bpmn/intellij/plugin/core/properties/PropertiesVisualizer.kt
@v20stepanenko
fix remove cascades prop in event type group
e5f5824
@v20stepanenko
fix selectList
89ea7fa
@v20stepanenko
fix cascades property update
dda2e0c
@v20stepanenko
fix cascades property update
ecaedbc
@v20stepanenko
Merge remote-tracking branch 'origin/send-event-task' into send-event…
09f8720 
…-task

# Conflicts:
#	bpmn-intellij-plugin-core/src/main/kotlin/com/valb3r/bpmn/intellij/plugin/core/properties/Util.kt
@v20stepanenko
fix properties visualise pads
4096bab
@v20stepanenko
fix test with properties visualizer pads
e48f98b
@valb3r valb3r merged commit cefdce1 into valb3r:master Aug 22, 2022
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@valb3r valb3r valb3r approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Send event task and listeners/triggers support
2 participants
@v20stepanenko @valb3r