We want to deprecate this function because it's no longer relevant for *RawThreadInfos without member permissions.
We're keeping it around to ensure existing migrations work as expected.
Depends on D12598
Differential D12599
[lib] Deprecate `updateRolesAndPermissions` atul on Jun 27 2024, 2:05 PM. Authored by Tags None Referenced Files
Subscribers
Details We want to deprecate this function because it's no longer relevant for *RawThreadInfos without member permissions. We're keeping it around to ensure existing migrations work as expected. Depends on D12598 This diff is just a rename to make it clear function shouldn't be used in future. Looked at all callsites and ensured there's nothing preventing function from being deprecated.
Diff Detail
Event TimelineComment Actions Is the idea here that we'll never need to run deprecatedUpdateRolesAndPermissions again when we change permissions, since we're not storing the permissions in the ThreadStore at all? And that permission checks in the future will be computed on-the-fly based on the user's role in the relevant thread and its ancestors? |