Page MenuHomePhabricator

Check for calendar before settings status
ClosedPublic

Authored by garoevans on Jun 16 2014, 7:10 PM.
Tags
None
Referenced Files
F14373824: D9577.id22952.diff
Fri, Dec 20, 8:56 PM
F14371379: D9577.id22951.diff
Fri, Dec 20, 6:56 PM
Unknown Object (File)
Fri, Dec 13, 12:37 AM
Unknown Object (File)
Wed, Dec 11, 2:36 PM
Unknown Object (File)
Mon, Dec 2, 12:51 PM
Unknown Object (File)
Sat, Nov 30, 1:01 PM
Unknown Object (File)
Mon, Nov 25, 2:37 PM
Unknown Object (File)
Nov 18 2024, 8:44 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).