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
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
Unknown Object (File)
Wed, Jun 19, 12:20 PM
Unknown Object (File)
Fri, Jun 14, 8:35 AM
Unknown Object (File)
Wed, Jun 12, 10:33 AM
Unknown Object (File)
Tue, Jun 11, 3:24 PM
View All 29 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.Wed, May 29, 10:24 PM

rebase

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

ready to land

This revision was landed with ongoing or failed builds.Wed, May 29, 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.Wed, May 29, 10:41 PM
Harbormaster completed remote builds in B29300: Diff 40770.Wed, May 29, 10:46 PM

Revision Contents
Changeset List

PathSize
lib/
shared/
4 lines

Diff 40771

View Options

lib/shared/thread-utils.js

Loading...