Skip to content

Implement Payload<String, Any?> changes#274

Merged
zambrovski merged 4 commits intodevelopfrom
feat/issue-268
Feb 5, 2026
Merged

Implement Payload<String, Any?> changes#274
zambrovski merged 4 commits intodevelopfrom
feat/issue-268

Conversation

@zambrovski
Copy link
Contributor

emaarco and others added 3 commits January 20, 2026 19:24
- Updated PayloadSupplier interface to Supplier<Map<String, Any?>>
- Updated all 14 command constructors to accept Map<String, Any?>
- Updated TaskHandler interface and implementations
- Updated UserTaskSupport to handle nullable values
- Backward compatible change

Fixes #268
Implement support for null values in PayloadSupplier
@zambrovski zambrovski added this to the 1.5 milestone Feb 5, 2026
@zambrovski zambrovski added the Prio: MUST Feature is essential for the milestone. label Feb 5, 2026
Copy link
Contributor Author

@zambrovski zambrovski left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you @emaarco

@zambrovski zambrovski merged commit 5c84a0e into develop Feb 5, 2026
4 checks passed
@zambrovski zambrovski deleted the feat/issue-268 branch February 5, 2026 16:39
@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

❌ Patch coverage is 26.66667% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.04%. Comparing base (f76f147) to head (3cdb1e2).
⚠️ Report is 5 commits behind head on develop.

Files with missing lines Patch % Lines
...pi/process/StartProcessByDefinitionAtElementCmd.kt 0.00% 2 Missing ⚠️
...ssengineapi/process/StartProcessByDefinitionCmd.kt 0.00% 2 Missing ⚠️
...ocessengineapi/process/StartProcessByMessageCmd.kt 0.00% 2 Missing ⚠️
...ters/processengineapi/correlation/SendSignalCmd.kt 0.00% 1 Missing ⚠️
...gineapi/decision/DecisionByRefEvaluationCommand.kt 0.00% 1 Missing ⚠️
...rs/processengineapi/task/CompleteTaskByErrorCmd.kt 0.00% 1 Missing ⚠️
...mcrafters/processengineapi/task/CompleteTaskCmd.kt 0.00% 1 Missing ⚠️
...rocessengineapi/task/support/AssignmentDetector.kt 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             develop     #274   +/-   ##
==========================================
  Coverage      46.04%   46.04%           
  Complexity        67       67           
==========================================
  Files             41       41           
  Lines            417      417           
  Branches           9        9           
==========================================
  Hits             192      192           
  Misses           220      220           
  Partials           5        5           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Prio: MUST Feature is essential for the milestone.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

As a user I want to pass payload to commands that contain null values

2 participants