feat: add enforceUniqueInstanceId option to orchestration and workflow instances - #1803
Conversation
9443f1a to
55b9fa3
Compare
|
thanks @matheusandre1 ! Can you modify the class WorkflowInstanceAlreadyExistsException.java so the docs make clear that this is throw not only for active workflows? with this new optin true it fails with any status |
55b9fa3 to
1e578cb
Compare
…w instances Signed-off-by: Matheus André <matheusandr2@gmail.com>
1e578cb to
2cf8c73
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1803 +/- ##
============================================
+ Coverage 77.15% 77.17% +0.02%
- Complexity 2317 2324 +7
============================================
Files 245 245
Lines 7186 7198 +12
Branches 750 750
============================================
+ Hits 5544 5555 +11
- Misses 1281 1283 +2
+ Partials 361 360 -1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Thanks @javier-aliaga! |
|
@holopin-bot @matheusandre1 sdk-badge Thank you! Here's a digital badge as a small token of appreciation. |
|
Congratulations @matheusandre1, the maintainer of this repository has issued you a badge! Here it is: https://holopin.io/claim/cmtr0q9me0003ld04417kjatd This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account. |
Description
Please explain the changes you've made
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will Closes: #1801
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: