Page MenuHomePhabricator

[lib] Batch BaseAutoJoinCommunityHandler
ClosedPublic

Authored by ashoat on Oct 2 2024, 11:58 PM.
Tags
None
Referenced Files
F3401143: D13583.id44849.diff
Mon, Dec 2, 10:44 AM
F3400984: D13583.id44851.diff
Mon, Dec 2, 9:56 AM
F3400503: D13583.id44850.diff
Mon, Dec 2, 7:29 AM
Unknown Object (File)
Tue, Nov 26, 4:19 AM
Unknown Object (File)
Sat, Nov 23, 6:27 AM
Unknown Object (File)
Sun, Nov 10, 11:46 AM
Unknown Object (File)
Sun, Nov 10, 10:07 AM
Unknown Object (File)
Sun, Nov 10, 6:26 AM
Subscribers

Details

Summary

This diff updates BaseAutoJoinCommunityHandler to work in batches of 5, and to avoid triggering a joinCommunity for a single community more than once.

Test Plan

Varun tested this in his local environment, where he has set up a reproduction of the production environment where we have a lot of communities tagged with Farcaster channels, and he tested signing in as a user with a lot of channels

Diff Detail

Repository
rCOMM Comm
Branch
ashoat/autojoin
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

not reviewing too closely because it's late

This revision is now accepted and ready to land.Oct 3 2024, 12:32 AM

Add sleep(1000) before joinCommunity. Testing on prod this avoided the lag issue

This revision was landed with ongoing or failed builds.Oct 3 2024, 12:41 AM
This revision was automatically updated to reflect the committed changes.