Paths

Table of Contentst

Differential D13583

[lib] Batch BaseAutoJoinCommunityHandler
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Dec 18, 8:38 AM

	Unknown Object (File)
	Wed, Dec 18, 8:38 AM

	Unknown Object (File)
	Wed, Dec 18, 8:38 AM

	Unknown Object (File)
	Wed, Dec 18, 8:38 AM

	Unknown Object (File)
	Wed, Dec 18, 8:37 AM

	Unknown Object (File)
	Mon, Dec 2, 10:44 AM

	Unknown Object (File)
	Mon, Dec 2, 9:56 AM

	Unknown Object (File)
	Mon, Dec 2, 7:29 AM

View All 47 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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

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

Herald added a subscriber: tomek. · View Herald TranscriptOct 2 2024, 11:58 PM

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

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

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

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.Oct 3 2024, 12:57 AM

Revision Contents
Changeset List

Path

Size

lib/

components/

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

175 lines

Diff 44851

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

Loading...