Page MenuHomePhabricator
Differential D9582

Only show profile status when calendar app installed
ClosedPublic
Actions

Authored by garoevans on Jun 16 2014, 9:17 PM.
Tags
None
Referenced Files
F13135918: D9582.id22964.diff
Thu, May 2, 11:21 AM
Unknown Object (File)
Mon, Apr 29, 7:59 PM
Unknown Object (File)
Wed, Apr 24, 11:03 PM
Unknown Object (File)
Tue, Apr 23, 6:54 PM
Unknown Object (File)
Sat, Apr 20, 7:06 PM
Unknown Object (File)
Tue, Apr 16, 6:43 AM
Unknown Object (File)
Tue, Apr 16, 12:27 AM
Unknown Object (File)
Thu, Apr 11, 10:06 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T5087: Disable "Status" field in profile when calendar is disabled
Commits
Restricted Diffusion Commit
Restricted Diffusion Commit
rPa10f969919a9: Revert "Query daemons across all hosts with `./bin/phd status --all`."
rP78635a15c108: Only show profile status when calendar app installed
Summary

If the calendar app is not installed we don't show the status.
Origianlly the idea was to only show the status if the viewer had access to
the app, but for display purposes this seems fine.

Fixes T5087

Test Plan

View with and without calendar installed

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

garoevans updated this revision to Diff 22964.Jun 16 2014, 9:17 PM
garoevans added a task: T5087: Disable "Status" field in profile when calendar is disabled.
garoevans retitled this revision from to Only show profile status when calendar app installed.
garoevans updated this object.
garoevans edited the test plan for this revision. (Show Details)
garoevans added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 16 2014, 9:17 PM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
Harbormaster completed remote builds in B1153: Diff 22964.Jun 16 2014, 9:18 PM
epriestley accepted this revision.Jun 16 2014, 9:18 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 16 2014, 9:18 PM
epriestley closed this revision.Jun 16 2014, 9:18 PM
epriestley updated this revision to Diff 22965.

Closed by commit rP78635a15c108 (authored by @garoevans, committed by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
people/
customfield/
5 lines

Diff 22965

View Options

src/applications/people/customfield/PhabricatorUserStatusField.php

Loading...
Phabricator · Built by Phacility