Page MenuHomePhabricator

Check for calendar before settings status
ClosedPublic

Authored by garoevans on Jun 16 2014, 7:10 PM.
Tags
None
Referenced Files
F14482467: D9577.id.diff
Mon, Dec 30, 9:16 AM
F14482341: D9577.diff
Mon, Dec 30, 8:25 AM
F14481736: D9577.diff
Mon, Dec 30, 5:02 AM
Unknown Object (File)
Thu, Dec 26, 2:39 PM
Unknown Object (File)
Thu, Dec 26, 9:33 AM
Unknown Object (File)
Mon, Dec 23, 4:44 AM
Unknown Object (File)
Sat, Dec 21, 9:17 AM
Unknown Object (File)
Fri, Dec 20, 8:56 PM

Details

Summary

We should not show the status line in the people hover card
if the calendar app has been uninstalled or is not available for the
current user.

Test Plan

View hover card with calendar installed and uninstalled.
Make sure I see the status at the correct time.

Diff Detail

Repository
rP Phabricator
Branch
T5370
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1147
Build 1147: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

garoevans retitled this revision from to Check for calendar before settings status.
garoevans updated this object.
garoevans edited the test plan for this revision. (Show Details)
garoevans added a reviewer: epriestley.
garoevans added a subscriber: chad.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 16 2014, 7:17 PM
epriestley updated this revision to Diff 22952.

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