Page MenuHomePhabricator
Differential D12713

[lib] make useJoinCommunity compatible with the auto join community handler
ClosedPublic
Actions

Authored by ginsu on Jul 9 2024, 11:27 PM.
Tags
None
Referenced Files
F3350430: D12713.diff
Fri, Nov 22, 10:30 PM
Unknown Object (File)
Wed, Nov 20, 3:42 AM
Unknown Object (File)
Wed, Nov 20, 3:41 AM
Unknown Object (File)
Wed, Nov 20, 3:10 AM
Unknown Object (File)
Sun, Nov 17, 5:17 PM
Unknown Object (File)
Fri, Nov 8, 10:39 PM
Unknown Object (File)
Fri, Nov 8, 7:14 AM
Unknown Object (File)
Fri, Nov 8, 7:13 AM
View All 80 Files
Subscribers
ashoat
tomek

Details

Reviewers
ashoat
inka
Commits
rCOMMc28089f95107: [lib] make useJoinCommunity compatible with the auto join community handler
Summary

This diff updates useJoinCommunity to be more agnostic so that this hook can be used for both invite links + auto join community handler

Depends on D12706

Test Plan

flow + confirmed that there were no regressions with the joining a community through accepitng an invite link user flow

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu created this revision.Jul 9 2024, 11:27 PM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptJul 9 2024, 11:27 PM
ginsu edited the summary of this revision. (Show Details)Jul 9 2024, 11:37 PM
ginsu added reviewers: ashoat, inka.
ginsu added a parent revision: D12706: [lib] introduce useJoinCommunity.
Harbormaster completed remote builds in B30256: Diff 42196.Jul 9 2024, 11:48 PM
ginsu requested review of this revision.Jul 9 2024, 11:48 PM
ginsu added a child revision: D12714: [native] update AutoJoinCommunityHandler to use useJoinCommunity hook.Jul 10 2024, 2:10 AM
ashoat accepted this revision.Jul 10 2024, 10:56 AM
This revision is now accepted and ready to land.Jul 10 2024, 10:56 AM
ginsu updated this revision to Diff 42219.Jul 10 2024, 11:56 PM

update

Harbormaster completed remote builds in B30277: Diff 42219.Jul 11 2024, 12:13 AM
ginsu updated this revision to Diff 42245.Jul 11 2024, 8:56 PM

rebase before landing

This revision was landed with ongoing or failed builds.Jul 11 2024, 9:01 PM
Closed by commit rCOMMc28089f95107: [lib] make useJoinCommunity compatible with the auto join community handler (authored by ginsu). · Explain Why
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMMc28089f95107: [lib] make useJoinCommunity compatible with the auto join community handler.
Harbormaster completed remote builds in B30297: Diff 42245.Jul 11 2024, 9:12 PM

Revision Contents
Changeset List

PathSize
lib/
shared/
23 lines

Diff 42219

View Options

lib/shared/community-utils.js

Loading...