Paths

Table of Contentst

Differential D12309

[lib][web] Extract logic checking add users permissions
ClosedPublic
Actions

Authored by inka on Tue, Jun 4, 2:03 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Jun 24, 3:58 PM

	Unknown Object (File)
	Fri, Jun 21, 6:23 PM

	Unknown Object (File)
	Fri, Jun 21, 6:26 AM

	Unknown Object (File)
	Thu, Jun 20, 9:08 AM

	Unknown Object (File)
	Thu, Jun 20, 4:44 AM

	Unknown Object (File)
	Mon, Jun 17, 10:24 AM

	Unknown Object (File)
	Mon, Jun 10, 9:51 PM

	Unknown Object (File)
	Mon, Jun 10, 9:00 PM

View All 25 Files

Subscribers

Details

Reviewers

Commits

rCOMM0c9ea4bbfdd1: [lib][web] Extract logic checking add users permissions

Summary

This can be reused on native

Test Plan

tested that in members modal the "Add members" button is displayed when it is supposed to and navigates to proper modals

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

inka created this revision.Tue, Jun 4, 2:03 AM

Herald added a subscriber: ashoat. · View Herald TranscriptTue, Jun 4, 2:03 AM

inka added a parent revision: D12237: [web] Navigate to invite links instead of add members modal.Tue, Jun 4, 2:03 AM

inka added a child revision: D12284: [native] Navigate to invite links instead of add members modal.

Harbormaster completed remote builds in B29425: Diff 40926.Tue, Jun 4, 2:19 AM

inka requested review of this revision.Tue, Jun 4, 2:19 AM

tomek accepted this revision.Tue, Jun 4, 2:45 AM

tomek added inline comments.

lib/permissions/add-users-permissions.js
11 ↗	(On Diff #40926)	How about renaming it to `AddUsersPermissions`?
18 ↗	(On Diff #40926)	I'm not sure if this name is good, because this hook returns more than just the permissions, but I don't have any better alternative.

This revision is now accepted and ready to land.Tue, Jun 4, 2:45 AM

Rename type

lib/permissions/add-users-permissions.js
18 ↗	(On Diff #40926)	I'm not 100% happy with it either, but I couldn't come up with anything better

Harbormaster completed remote builds in B29436: Diff 40943.Tue, Jun 4, 5:05 AM

Closed by commit rCOMM0c9ea4bbfdd1: [lib][web] Extract logic checking add users permissions (authored by inka). · Explain WhyTue, Jun 4, 8:31 AM

This revision was automatically updated to reflect the committed changes.

inka added a commit: rCOMM0c9ea4bbfdd1: [lib][web] Extract logic checking add users permissions.

Revision Contents
Changeset List

Path

Size

lib/

permissions/

add-users-permissions.js

41 lines

web/

modals/

threads/

members/

members-modal.react.js

29 lines

Diff 40965

lib/permissions/add-users-permissions.js

Loading...

web/modals/threads/members/members-modal.react.js

Loading...