elsa-workflows / elsa-workflows/elsa-core

BPMN document PUT: optionally assert Data in matchesExpected for symmetry with EF MatchesLoadedSnapshot

Open
#8,097 0 comments 0 reactions 0 assignees View on GitHub
elsa 3 enhancement prio low triaged
Dominant language
C#
Stars
7.9k
Forks
1.5k
Avg merge
15h 22m
Merged PRs (30d)
114

Description

## Summary

Follow-up from https://github.com/elsa-workflows/elsa-core/pull/8092 (issue #8064).

Architect + Crew Lead waived Greptile residual P1s on tip `2889d26`: BPMN `matchesExpected` scopes to id/version/StringData/Name/Description/IsLatest/ETag. EF `MatchesLoadedSnapshot` already includes `Data` (variables/options/custom props); Memory CAS is under `store.Sync`. No silent overwrite for landed stores.

Optional 3.9 improvement: also assert `Data` (or equivalent) in BPMN `matchesExpected` for **symmetry** with EF — not a correctness fix for current stores.

## Out of scope for 3.8 / already merged

Do not reopen #8092. This is Backlog / 3.9 only.

Contributor guide

Open the contributing guide

Research direction

Start with PR #8092 and the BPMN document PUT path, then compare its matchesExpected scope with EF MatchesLoadedSnapshot and the Memory store.Sync behavior. Confirm whether Data can be included without changing current-store correctness; done means the BPMN comparison optionally asserts Data for 3.9 symmetry without reopening #8092.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
databases
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.