Page MenuHomePhabricator
Differential D12021

Hide revision content if author has not signed required legal documents
ClosedPublic
Actions

Authored by epriestley on Mar 9 2015, 12:36 AM.
Tags
None
Referenced Files
F13140166: D12021.diff
Fri, May 3, 3:32 AM
Unknown Object (File)
Mon, Apr 29, 3:22 PM
Unknown Object (File)
Wed, Apr 24, 11:14 PM
Unknown Object (File)
Wed, Apr 17, 6:10 PM
Unknown Object (File)
Tue, Apr 16, 2:12 AM
Unknown Object (File)
Thu, Apr 4, 8:24 PM
Unknown Object (File)
Thu, Apr 4, 2:07 PM
Unknown Object (File)
Mar 31 2024, 10:27 AM
View All Files
Subscribers
epriestley
Tokens
"Grey Medal" token, awarded by btrahan.

Details

Reviewers
btrahan
Maniphest Tasks
T7446: When a revision requires a CLA, hide the text of the diff
Commits
Restricted Diffusion Commit
rPb7be4ccca187: Hide revision content if author has not signed required legal documents
Summary

Fixes T7446.

Test Plan

Screen_Shot_2015-03-08_at_5.33.12_PM.png (1×1 px, 214 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 28931.Mar 9 2015, 12:36 AM
epriestley retitled this revision from to Hide revision content if author has not signed required legal documents.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T7446: When a revision requires a CLA, hide the text of the diff.
Herald added a subscriber: epriestley. · View Herald TranscriptMar 9 2015, 12:36 AM
btrahan accepted this revision.Mar 9 2015, 5:16 PM
btrahan edited edge metadata.
btrahan added inline comments.
src/applications/differential/controller/DifferentialRevisionViewController.php
452

looks like $comment_view gets appended twice in this code path

This revision is now accepted and ready to land.Mar 9 2015, 5:16 PM
btrahan awarded a token.Mar 9 2015, 5:17 PM
epriestley added inline comments.Mar 9 2015, 5:20 PM
src/applications/differential/controller/DifferentialRevisionViewController.php
452

Whoops, good catch. I've been scrolling past double comments all weekend.

epriestley updated this revision to Diff 28937.Mar 9 2015, 5:20 PM
epriestley edited edge metadata.
  • One comment view is enough comment views.
Closed by commit rPb7be4ccca187: Hide revision content if author has not signed required legal documents (authored by epriestley, committed by epriestley). · Explain WhyMar 9 2015, 5:28 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
controller/
46 lines
docs/
user/
userguide/
4 lines

Diff 28944

View Options

src/applications/differential/controller/DifferentialRevisionViewController.php

Loading...
View Options

src/docs/user/userguide/legalpad.diviner

Loading...
Phabricator · Built by Phacility