Page MenuHomePhabricator

[lib] Change `decodeMinimallyEncodedRawThreadInfo` argument type to `RawThreadInfoWithMemberPermissions`
AbandonedPublic

Authored by atul on Tue, Jun 25, 9:54 AM.

Details

Reviewers
ashoat
ginsu
tomek
Summary

Brings us from 34 to 29 flow issues.


Depends on D12557

Test Plan

flow + looked at callsites to ensure things are as expected.

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Tue, Jun 25, 10:07 AM
Harbormaster failed remote builds in B29923: Diff 41697!
atul requested review of this revision.Wed, Jun 26, 9:36 AM
atul added inline comments.
lib/shared/redux/update-roles-and-permissions.js
13

This import will be removed.

Abandoning in favor of stack starting with D12594 because lots of things have changed and there's not really 1:1 correspondence with old stack and new stack so I don't think it makes sense to update in place, change a bunch of child/parent diff relationships, etc.