Page MenuHomePhabricator

Check for calendar before settings status
ClosedPublic

Authored by garoevans on Jun 16 2014, 7:10 PM.
Tags
None
Referenced Files
F13107273: D9577.id.diff
Sat, Apr 27, 10:27 PM
Unknown Object (File)
Sat, Apr 27, 3:48 AM
Unknown Object (File)
Wed, Apr 24, 10:49 PM
Unknown Object (File)
Tue, Apr 23, 6:54 PM
Unknown Object (File)
Sat, Apr 20, 7:05 PM
Unknown Object (File)
Thu, Apr 11, 10:03 AM
Unknown Object (File)
Tue, Apr 9, 5:04 AM
Unknown Object (File)
Sat, Mar 30, 5:34 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).