Page MenuHomePhabricator

[lib] Add blob fields to DMOperationResult
ClosedPublic

Authored by bartek on Sep 20 2024, 6:13 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 1, 7:55 AM
Unknown Object (File)
Thu, Nov 28, 9:00 PM
Unknown Object (File)
Nov 20 2024, 10:36 AM
Unknown Object (File)
Nov 20 2024, 10:35 AM
Unknown Object (File)
Nov 20 2024, 10:35 AM
Unknown Object (File)
Nov 20 2024, 10:35 AM
Unknown Object (File)
Nov 1 2024, 5:37 AM
Unknown Object (File)
Oct 30 2024, 11:27 PM
Subscribers

Details

Summary

This will be needed for ENG-9288.
DM ops will return blob hashes that need establishing / removing their holders.

Test Plan

Flow. Logic added in further diffs

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.Sep 20 2024, 6:42 AM
kamil added inline comments.
lib/types/dm-ops.js
490–493 ↗(On Diff #44366)

you need to add $ReadOnly< {} > - spreading remove this property

This revision is now accepted and ready to land.Sep 20 2024, 7:40 AM
This revision was automatically updated to reflect the committed changes.