Details
Details
- Maniphest Tasks
- T208254: Legalpad access controls are confusing and seemingly broken
- Reviewers
JeanFred Dzahn mmodell - Commits
- rPHABaec846f37b26: Fix the check for is_admin in LegalpadDocumentRequireSignatureTransaction
- Patch without arc
- git checkout -b D1123 && curl -L https://phabricator.wikimedia.org/D1123?download=true | git apply
tested on my local phab instance
Diff Detail
Diff Detail
- Repository
- rPHAB Phabricator
- Branch
- wmf/stable
- Lint
Lint OK - Unit
Unit Tests OK - Build Status
Buildable 3156 Build 5284: arc lint + arc unit
Event Timeline
Comment Actions
committed rPHABaec846f37b26: Fix the check for is_admin in LegalpadDocumentRequireSignatureTransaction and cherry-picked on phab1001