Paths

Table of Contentst

Differential D13583

[lib] Batch BaseAutoJoinCommunityHandler
ClosedPublic
Actions

Authored by ashoat on Wed, Oct 2, 11:58 PM.

Tags

None

Referenced Files

	F2935336: D13583.diff
	Thu, Oct 10, 7:25 AM

	F2934201: D13583.id44849.diff
	Thu, Oct 10, 5:26 AM

	F2929617: D13583.id44851.diff
	Wed, Oct 9, 3:44 PM

	F2929106: D13583.id44850.diff
	Wed, Oct 9, 11:40 AM

	Unknown Object (File)
	Tue, Oct 8, 1:03 AM

	Unknown Object (File)
	Thu, Oct 3, 3:15 PM

	Unknown Object (File)
	Thu, Oct 3, 5:50 AM

	Unknown Object (File)
	Thu, Oct 3, 5:47 AM

View All 15 Files

Subscribers

Details

Reviewers

varun
will

Commits

rCOMM4c5fc7883f77: [lib] Batch BaseAutoJoinCommunityHandler

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

Branch

ashoat/autojoin

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Wed, Oct 2, 11:58 PM

Herald added a subscriber: tomek. · View Herald TranscriptWed, Oct 2, 11:58 PM

Harbormaster completed remote builds in B32032: Diff 44849.Thu, Oct 3, 12:15 AM

ashoat requested review of this revision.Thu, Oct 3, 12:15 AM

not reviewing too closely because it's late

This revision is now accepted and ready to land.Thu, Oct 3, 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.Thu, Oct 3, 12:41 AM

Closed by commit rCOMM4c5fc7883f77: [lib] Batch BaseAutoJoinCommunityHandler (authored by ashoat). · Explain Why

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM4c5fc7883f77: [lib] Batch BaseAutoJoinCommunityHandler.

Harbormaster completed remote builds in B32033: Diff 44850.Thu, Oct 3, 12:57 AM

Revision Contents
Changeset List

Path

Size

lib/

components/

base-auto-join-community-handler.react.js

173 lines

Diff 44849

lib/components/base-auto-join-community-handler.react.js

Loading...