Page MenuHomePhabricator

Hide "Availability" and "Calendar" on user profiles for disabled users
ClosedPublic

Authored by epriestley on Mar 28 2019, 11:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Feb 9, 7:30 AM
Unknown Object (File)
Sun, Feb 9, 7:29 AM
Unknown Object (File)
Sun, Feb 9, 7:29 AM
Unknown Object (File)
Sun, Feb 9, 7:29 AM
Unknown Object (File)
Tue, Jan 28, 6:43 PM
Unknown Object (File)
Sun, Jan 26, 11:17 PM
Unknown Object (File)
Sat, Jan 25, 4:28 AM
Unknown Object (File)
Sat, Jan 25, 4:28 AM
Subscribers
None

Details

Summary

See downstream https://phabricator.wikimedia.org/T138723. That suggestion is a little light on details, but I basically agree that showing "Availability: Available" on disabled user profiles is kind of questionable/misleading.

Just hide event information on disabled profiles, since this doesn't seem worth building a special "Availability: Who Knows, They Are Disabled, Good Luck" disabled state for.

Test Plan

Looked at disabled and non-disabled user profiles, saw Calendar stuff only on the former.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable