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

Process Definition event should not override metadata, but merge it #2177

Closed
fjtirado opened this issue Jan 20, 2025 · 0 comments · Fixed by #2178
Closed

Process Definition event should not override metadata, but merge it #2177

fjtirado opened this issue Jan 20, 2025 · 0 comments · Fixed by #2178
Assignees
@fjtirado

Comments

@fjtirado
Copy link
Contributor

fjtirado commented Jan 20, 2025 •
edited
Loading

Description

Currently, if ProcessDefinition metadata is updated, the old metadata is lost.

Implementation ideas

Implements Mergemap method
@fjtirado fjtirado self-assigned this Jan 20, 2025
fjtirado added a commit to fjtirado/kogito-apps that referenced this issue Jan 20, 2025
@fjtirado
[Fix apache#2177]
c54fe47 
Fix apache#2177
@fjtirado fjtirado mentioned this issue Jan 20, 2025
Merged
fjtirado added a commit to fjtirado/kogito-apps that referenced this issue Jan 20, 2025
@fjtirado
[Fix apache#2177] Merge metadata in ProcessEventDefinition
0123416 
Fix apache#2177
fjtirado added a commit to fjtirado/kogito-apps that referenced this issue Jan 20, 2025
@fjtirado
[Fix apache#2177] Merge metadata in ProcessEventDefinition
0cd7c8f 
Fix apache#2177
fjtirado added a commit to fjtirado/kogito-apps that referenced this issue Jan 20, 2025
@fjtirado
[Fix apache#2177] Partial rollback
57b1c5a
fjtirado added a commit to fjtirado/kogito-apps that referenced this issue Jan 20, 2025
@fjtirado
[Fix apache#2177] Partial rollback
f87757b
fjtirado added a commit to fjtirado/kogito-apps that referenced this issue Jan 20, 2025
@fjtirado
[Fix apache#2177] Partial rollback
52aa88c
fjtirado added a commit to fjtirado/kogito-apps that referenced this issue Jan 20, 2025
@fjtirado
[Fix apache#2177] Partial rollback
a33ccfd
fjtirado added a commit to fjtirado/kogito-apps that referenced this issue Jan 21, 2025
@fjtirado
[Fix apache#2177] Partial rollback
5d68dcb
fjtirado added a commit to fjtirado/kogito-apps that referenced this issue Jan 21, 2025
@fjtirado
[Fix apache#2177] Partial rollback
28d2437
fjtirado added a commit that referenced this issue Jan 22, 2025
@fjtirado
[Fix #2177] Merge ProcessDefinitionEvent metadata (#2178)
5c3f01b 
* [Fix #2177] Merge metadata in ProcessEventDefinition

Fix #2177

* [Fix #2177] Partial rollback
gmunozfe added a commit to gmunozfe/kogito-apps that referenced this issue Jan 27, 2025
@gmunozfe
[Fix apache#2177] Add test for better coverage
b368764
@gmunozfe gmunozfe mentioned this issue Jan 27, 2025
Merged
7 tasks
rgdoliveira pushed a commit to rgdoliveira/kogito-apps that referenced this issue Jan 27, 2025
@fjtirado @rgdoliveira
[Fix apache#2177] Merge ProcessDefinitionEvent metadata (apache#2178)
497e498 
* [Fix apache#2177] Merge metadata in ProcessEventDefinition

Fix apache#2177

* [Fix apache#2177] Partial rollback
@rgdoliveira rgdoliveira mentioned this issue Jan 27, 2025
Merged
fjtirado pushed a commit that referenced this issue Jan 28, 2025
@gmunozfe
[Fix #2177] Add test for better coverage (#2180)
03210e5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fjtirado fjtirado

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Fix #2177] Merge ProcessDefinitionEvent metadata fjtirado/kogito-apps
1 participant
@fjtirado