Page MenuHomePhabricator
Differential D12150

[lib] Make `checkOnlyViewerBlock` argument of `threadIsWithBlockedUserOnly` required
ClosedPublic
Actions

Authored by atul on May 21 2024, 8:04 AM.
Tags
None
Referenced Files
F2159278: D12150.id40770.diff
Mon, Jul 1, 6:37 AM
F2159261: D12150.id40769.diff
Mon, Jul 1, 6:31 AM
F2159064: D12150.id40480.diff
Mon, Jul 1, 6:06 AM
F2154251: D12150.id40769.diff
Sun, Jun 30, 6:52 PM
Unknown Object (File)
Fri, Jun 21, 5:28 AM
Unknown Object (File)
Fri, Jun 21, 5:28 AM
Unknown Object (File)
Fri, Jun 21, 5:28 AM
Unknown Object (File)
Thu, Jun 20, 4:45 AM
View All 33 Files
Subscribers
None

Details

Reviewers
ashoat
ginsu
tomek
varun
Commits
rCOMMe31838429e78: [lib] Make `checkOnlyViewerBlock` argument of `threadIsWithBlockedUserOnly`…
Summary

I want to add another skipMemberAdminRoleCheck argument to this function. Can't have two optional arguments, so going to make them both required.

Only two callsites to threadIsWithBlockedUserOnly and they're pretty straightforward.

Depends on D12149

Test Plan

Trusting flow here.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.May 21 2024, 8:04 AM
atul published this revision for review.May 21 2024, 8:04 AM
atul added a child revision: D12154: [lib] Add `skipMemberAdminRoleCheck` argument to `threadIsWithBlockedUserOnly`.May 21 2024, 8:27 AM
Harbormaster completed remote builds in B29113: Diff 40480.May 21 2024, 8:50 AM
varun resigned from this revision.May 22 2024, 7:41 AM

i'm probably not a good reviewer of this code

ashoat accepted this revision.May 24 2024, 2:51 AM
This revision is now accepted and ready to land.May 24 2024, 2:51 AM
atul updated this revision to Diff 40769.May 29 2024, 10:24 PM

rebase

atul updated this revision to Diff 40770.May 29 2024, 10:27 PM

ready to land

This revision was landed with ongoing or failed builds.May 29 2024, 10:35 PM
Closed by commit rCOMMe31838429e78: [lib] Make `checkOnlyViewerBlock` argument of `threadIsWithBlockedUserOnly`… (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMMe31838429e78: [lib] Make `checkOnlyViewerBlock` argument of `threadIsWithBlockedUserOnly`….
Harbormaster completed remote builds in B29299: Diff 40769.May 29 2024, 10:41 PM
Harbormaster completed remote builds in B29300: Diff 40770.May 29 2024, 10:46 PM

Revision Contents
Changeset List

PathSize
lib/
shared/
4 lines

Diff 40771

View Options

lib/shared/thread-utils.js

Loading...