Skip to content

Fix remote write merging and duplicate function permissions#93

Open
mohamedeltawel wants to merge 2 commits into
devfrom
fix/MergingAlgorithm
Open

Fix remote write merging and duplicate function permissions#93
mohamedeltawel wants to merge 2 commits into
devfrom
fix/MergingAlgorithm

Conversation

@mohamedeltawel

Copy link
Copy Markdown
Member

Summary

  • Fix remote partial merge behavior so unchanged existing items without write-check fields are preserved instead of failing the write.
  • Prevent remote writes from appending new items that are not allowed through the merge path.
  • Merge duplicate local feature function permissions with allowed-wins semantics:
    • a later allowed operation upgrades a previously not-allowed operation
    • a later not-allowed declaration does not downgrade an operation already allowed by another use case
  • Add regression tests for collection merge behavior, DeviceConfiguration partial writes, and duplicate function permission merging.

@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 93.728% (+0.04%) from 93.691% — fix/MergingAlgorithm into dev

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants