Paths

Table of Contentst

Differential D12285

[native] Stop navigating to Add Members from community creation
ClosedPublic
Actions

Authored by inka on Jun 3 2024, 8:01 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Nov 16, 2:34 AM

	Unknown Object (File)
	Sun, Nov 10, 11:56 PM

	Unknown Object (File)
	Sun, Nov 10, 11:42 PM

	Unknown Object (File)
	Sun, Nov 10, 9:32 PM

	Unknown Object (File)
	Sun, Nov 10, 6:35 PM

	Unknown Object (File)
	Sun, Nov 10, 3:56 PM

	Unknown Object (File)
	Oct 15 2024, 2:22 AM

	Unknown Object (File)
	Oct 11 2024, 7:31 AM

View All 60 Files

Subscribers

Details

Reviewers

tomek
kamil

Commits

rCOMM4b5b50a21e6d: [native] Stop navigating to Add Members from community creation

Summary

issue: ENG-8246

Test Plan

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

inka created this revision.Jun 3 2024, 8:01 AM

Herald added a subscriber: ashoat. · View Herald TranscriptJun 3 2024, 8:01 AM

inka added a parent revision: D12284: [native] Navigate to invite links instead of add members modal.Jun 3 2024, 8:01 AM

inka edited the test plan for this revision. (Show Details)

inka attached a referenced file: F1942676: Screen Recording 2024-06-03 at 16.53.15.mov. (Show Details)

inka added inline comments.Jun 3 2024, 8:04 AM

native/community-creation/community-configuration.react.js
43 ↗	(On Diff #40871)	We cannot remove props because of react navigation. We do the same in other components, for example `AppsDirectory`
107 ↗	(On Diff #40871)	Before, we would navigate to `CommunityCreationMembersRouteName`, where upon pressing "Add" or "Cancel" (the only two buttons allowing us to leave the screen) `navigateToThread` would be called. We want to do this right away, since we don't want to allow adding users to community roots

Harbormaster completed remote builds in B29379: Diff 40871.Jun 3 2024, 8:18 AM

inka requested review of this revision.Jun 3 2024, 8:18 AM

inka added a child revision: D12286: [native] Fetch invite links when entering thread settings.Jun 3 2024, 8:34 AM

ashoat added inline comments.Jun 3 2024, 10:01 AM

native/community-creation/community-configuration.react.js
30 ↗	(On Diff #40871)	Is this route used anywhere else, or should it be deleted?

tomek accepted this revision.Jun 4 2024, 1:09 AM

tomek added inline comments.

native/community-creation/community-configuration.react.js
106–107 ↗	(On Diff #40871)	Is it possible that `communityThreadInfo` will still be null here? Should we navigate conditionally, only when the thread info is present?

This revision is now accepted and ready to land.Jun 4 2024, 1:09 AM

inka mentioned this in D12306: [native] Remove unused componenet.Jun 4 2024, 1:15 AM

inka added inline comments.Jun 4 2024, 1:16 AM

native/community-creation/community-configuration.react.js
30 ↗	(On Diff #40871)	We can remove it D12306

Check if thread info is present

Harbormaster completed remote builds in B29423: Diff 40924.Jun 4 2024, 1:56 AM

Closed by commit rCOMM4b5b50a21e6d: [native] Stop navigating to Add Members from community creation (authored by inka). · Explain WhyJun 4 2024, 8:31 AM

This revision was automatically updated to reflect the committed changes.

inka added a commit: rCOMM4b5b50a21e6d: [native] Stop navigating to Add Members from community creation.

inka mentioned this in rCOMMdbf70d0f8675: [native] Remove unused componenet.

Revision Contents
Changeset List

Path

Size

native/

community-creation/

community-configuration.react.js

40 lines

Diff 40966

native/community-creation/community-configuration.react.js

Loading...