Page MenuHomePhabricator

Use viwer-based checks for application visibility when rendering home elements
ClosedPublic

Authored by epriestley on Mar 15 2014, 3:30 PM.
Tags
None
Referenced Files
F14757465: D8545.id.diff
Wed, Jan 22, 1:20 AM
F14752931: D8545.id20280.diff
Tue, Jan 21, 1:56 PM
Unknown Object (File)
Tue, Jan 21, 11:37 AM
Unknown Object (File)
Tue, Jan 21, 8:59 AM
Unknown Object (File)
Thu, Dec 26, 5:25 AM
Unknown Object (File)
Tue, Dec 24, 8:45 PM
Unknown Object (File)
Dec 23 2024, 4:20 AM
Unknown Object (File)
Dec 22 2024, 4:57 PM
Subscribers

Details

Summary

Fixes T4619. Currently, even if a viewer can't see Maniphest, they'll still see empty panels on the home page. These panels will always be empty so there's no real policy violation, but it's confusing.

Longer term, dashboards should fix this.

Test Plan

Viewed home page with a user with and without permissions on the apps.

Diff Detail

Repository
rP Phabricator
Branch
apppolicy
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

epriestley retitled this revision from to Use viwer-based checks for application visibility when rendering home elements.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 15 2014, 3:59 PM
epriestley updated this revision to Diff 20280.

Closed by commit rPb3b0ef3647e8 (authored by @epriestley).