Paths

Table of Contentst

-
native/roles/
-
roles/
-
create-roles-header-right-button.react.js

Differential D8430

[native] Disable headerRight when the role name is empty
ClosedPublic
Actions

Authored by rohan on Jul 6 2023, 9:28 AM.

Tags

None

Referenced Files

	F5337971: D8430.id.diff
	Wed, Apr 9, 1:00 AM

	F5335521: D8430.id28866.diff
	Tue, Apr 8, 11:24 PM

	F5334985: D8430.id29383.diff
	Tue, Apr 8, 11:08 PM

	F5334783: D8430.id29322.diff
	Tue, Apr 8, 11:02 PM

	F5334470: D8430.id28440.diff
	Tue, Apr 8, 10:53 PM

	F5332274: D8430.id28490.diff
	Tue, Apr 8, 10:03 PM

	Unknown Object (File)
	Wed, Apr 2, 7:32 PM

	Unknown Object (File)
	Wed, Apr 2, 10:23 AM

Subscribers

Details

Reviewers

atul
ginsu

Commits

rCOMMeeeb249e17e8: [native] Disable headerRight when the role name is empty

Summary

In an effort to prevent the user from reaching a state that would result in an error (and just a general better UX overall), we should disable the headerRight 'Create' button when the role name is empty. We should allow a user to create a role without any selected permissions because on the keyserver, we add a set of guaranteed permissions anyways, so we won't end up with a case of a role with no permissions.

Error handling for duplicate role names / etc will be done in a later diff.

Part of ENG-4178

Depends on D8428

Test Plan

Verified that the headerRight component is disabled when it should be, and enabled when the role name is non-empty.

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

rohan created this revision.Jul 6 2023, 9:28 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJul 6 2023, 9:28 AM

rohan edited the summary of this revision. (Show Details)Jul 6 2023, 9:31 AM

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

rohan attached a referenced file: F623533: Simulator Screen Recording - iPhone 14 Pro - 2023-07-06 at 12.25.55.mp4. (Show Details)

rohan added a child revision: D8431: [native] Display loading indicator while role creation is in progress.Jul 6 2023, 9:40 AM

Harbormaster completed remote builds in B20764: Diff 28440.Jul 6 2023, 9:45 AM

rohan requested review of this revision.Jul 6 2023, 9:45 AM

Rebase on D8391 and D8420

Harbormaster completed remote builds in B20805: Diff 28490.Jul 7 2023, 10:15 AM

looks great!

This revision is now accepted and ready to land.Jul 12 2023, 8:01 PM

Rebase

Harbormaster completed remote builds in B21066: Diff 28866.Jul 19 2023, 3:09 PM

Rebase

Harbormaster completed remote builds in B21410: Diff 29322.Jul 31 2023, 5:23 PM

Closed by commit rCOMMeeeb249e17e8: [native] Disable headerRight when the role name is empty (authored by rohan). · Explain WhyJul 31 2023, 6:14 PM

This revision was automatically updated to reflect the committed changes.

rohan added a commit: rCOMMeeeb249e17e8: [native] Disable headerRight when the role name is empty.

Revision Contents
Changeset List

Path

Size

native/

roles/

create-roles-header-right-button.react.js

31 lines

Diff 28440

native/roles/create-roles-header-right-button.react.js

Loading...