Page MenuHomePhabricator

[web] Add manage invite links menu item
ClosedPublic

Authored by tomek on Jun 23 2023, 8:52 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 7, 4:37 PM
Unknown Object (File)
Tue, May 7, 9:46 AM
Unknown Object (File)
Tue, May 7, 9:46 AM
Unknown Object (File)
Tue, May 7, 9:46 AM
Unknown Object (File)
Apr 3 2024, 9:26 AM
Unknown Object (File)
Apr 3 2024, 9:26 AM
Unknown Object (File)
Apr 3 2024, 9:26 AM
Unknown Object (File)
Apr 3 2024, 9:25 AM
Subscribers

Details

Summary

The new item is displayed conditionally based on the permissions and uses an SVG icon.

menu-text.png (288×454 px, 20 KB)

Depends on D8306

Test Plan

Check if the new item is displayed for communities where user has permission.
Check if invite link button is displayed only when invite link exists.
Check if three dots button is displayed only when there's a menu item.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Jun 23 2023, 9:14 AM
Harbormaster failed remote builds in B20472: Diff 28054!
tomek requested review of this revision.Jun 26 2023, 4:13 AM

Shouldn't we return null when items is an empty array, as before?

Shouldn't we return null when items is an empty array, as before?

We discussed this offline and the code doesn't need to be changed. The menu button will not render because Menu component makes sure of that, but the container will, which allows for all long community names to end in the same place (whether the button is present or not), as had been requested in the designs.

This revision is now accepted and ready to land.Jun 27 2023, 1:11 AM
This revision now requires review to proceed.Jun 29 2023, 6:00 AM
web/invite-links/invite-links-menu.react.js
51 ↗(On Diff #28054)

I guess we should be consistent with native and use Manage invite links

ashoat added inline comments.
web/invite-links/invite-links-menu.react.js
51 ↗(On Diff #28054)

Yes please!

59 ↗(On Diff #28054)

This should just be "Invite link"

This revision is now accepted and ready to land.Jul 2 2023, 1:43 PM
This revision was automatically updated to reflect the committed changes.