Page MenuHomePhabricator

Check for calendar before settings status
ClosedPublic

Authored by garoevans on Jun 16 2014, 7:10 PM.
Tags
None
Referenced Files
F14063894: D9577.diff
Mon, Nov 18, 8:44 PM
F14003395: D9577.id22952.diff
Sat, Oct 26, 5:52 AM
F13982387: D9577.diff
Oct 19 2024, 10:51 PM
F13968115: D9577.id22951.diff
Oct 16 2024, 6:45 PM
Unknown Object (File)
Sep 19 2024, 10:43 AM
Unknown Object (File)
Sep 6 2024, 6:40 AM
Unknown Object (File)
Aug 8 2024, 7:18 PM
Unknown Object (File)
Aug 4 2024, 12:57 AM

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).