[lib] Deprecate `updateRolesAndPermissions` migration utility
AbandonedPublic
Actions

Authored by atul on Jun 26 2024, 4:14 PM.

Details

Reviewers

ashoat
ginsu
tomek

Summary

We keep this utility around so existing migrations continue to work. However, we will no longer have member permissions in the future so this utility is no longer relevant. Portions of it (eg ThreadCurrentUser) could be repurposed in the future.

This takes us from 28 -> 16 flow issues.

Depends on D12574

Test Plan

No logic changes just rename + flow suppression, close reading.

Diff Detail

Repository

rCOMM Comm

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

remove extraneous import

Harbormaster returned this revision to the author for changes because remote builds failed.Jun 26 2024, 4:27 PM

Harbormaster failed remote builds in B29931: Diff 41708!

Harbormaster failed remote builds in B29932: Diff 41709!

atul requested review of this revision.Jun 26 2024, 4:37 PM

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.