remove `change.operation` - Githubissues

opral / lix-sdk

1 stars 0 forks source link

remove `change.operation` #41

Closed samuelstroschein closed 2 weeks ago

samuelstroschein commented 2 months ago

Context

change.operation was introduced mainly for "lix (via the host) and lix apps need awareness if something has been inserted, updated, or deleted to render the correct diffs" see https://github.com/opral/monorepo/pull/3046#issuecomment-2273719740.

The assumption that lix (apps) needs a first level concept if something was a create, update, or delete does not seem to hold true:

lix is not responsible for applying changes (a plugin is)
lix is not responsible for showing diffs (a plugin/UI component is)
given that lix is neither responsible for applying nor displaying (diffs of) changes, a first-level operation concept does not seem necessary.

Proposal

Remove change.operation for now. It can be re-introduced at a later point if necessary.