Page MenuHomePhabricator

[lib] Introduce new permissions
AcceptedPublic

Authored by tomek on Wed, Mar 19, 7:46 AM.

Details

Reviewers
kamil
ashoat
Summary
Test Plan

Tested in combination with the next diff. Checked if the permissions appear in the manage role UI.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek requested review of this revision.Wed, Mar 19, 8:05 AM

I don't remember all the places where we need to add a new permission. Wondering if you modelled this on another diff that introduced a different permission – if so, would be helpful to share a link to that diff.

This revision is now accepted and ready to land.Wed, Mar 19, 9:38 AM
lib/types/thread-permission-types.js
448

Typo: capital L