Page MenuHomePhabricator

[lib] Merge viewerID into ProcessDMOperationUtilities
ClosedPublic

Authored by ashoat on Sep 20 2024, 6:30 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 6, 9:03 AM
Unknown Object (File)
Sat, Mar 15, 3:41 AM
Unknown Object (File)
Feb 24 2025, 8:40 PM
Unknown Object (File)
Feb 24 2025, 11:09 AM
Unknown Object (File)
Feb 20 2025, 9:14 AM
Unknown Object (File)
Feb 18 2025, 2:26 PM
Unknown Object (File)
Feb 16 2025, 3:45 PM
Unknown Object (File)
Feb 15 2025, 6:39 PM
Subscribers
None

Details

Summary

Following feedback in D13391.

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Branch
ashoat/relationship
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil added inline comments.
lib/shared/dm-ops/process-dm-ops.js
86โ€“89

is it needed? Why can't we keep using baseUtilities?

This revision is now accepted and ready to land.Sep 20 2024, 7:12 AM
lib/shared/dm-ops/process-dm-ops.js
86โ€“89

Flow needed this