Page MenuHomePhabricator
Differential D17317

Clarify that "account.editable" no longer extends to profile pictures
ClosedPublic
Actions

Authored by epriestley on Feb 6 2017, 1:02 PM.
Tags
None
Referenced Files
F18767709: D17317.diff
Tue, Oct 7, 11:54 PM
F18761019: D17317.id.diff
Mon, Oct 6, 11:55 AM
F18678121: D17317.id41647.diff
Thu, Sep 25, 10:22 PM
F18630121: D17317.id41646.diff
Sep 16 2025, 10:49 AM
F18601717: D17317.diff
Sep 13 2025, 1:17 PM
F18504708: D17317.diff
Sep 4 2025, 11:59 PM
F18093164: D17317.id41646.diff
Aug 7 2025, 3:58 PM
F18091329: D17317.id41647.diff
Aug 6 2025, 9:08 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T12216: User can change userpic when account.editable is false (bad config description or restrictions mistake)
Commits
rP3d44208e4f65: Clarify that "account.editable" no longer extends to profile pictures
Summary

Fixes T12216. I'd like to remove this option eventually, but just narrow its scope in the config description for now.

Test Plan

Read config description.

Diff Detail

Repository
rP Phabricator
Branch
editable1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15548
Build 20505: Run Core Tests
Build 20504: arc lint + arc unit

Event Timeline

epriestley created this revision.Feb 6 2017, 1:02 PM
Harbormaster completed remote builds in B15548: Diff 41646.Feb 6 2017, 1:03 PM
chad accepted this revision.Feb 6 2017, 3:32 PM
This revision is now accepted and ready to land.Feb 6 2017, 3:32 PM
Closed by commit rP3d44208e4f65: Clarify that "account.editable" no longer extends to profile pictures (authored by epriestley, committed by epriestley). · Explain WhyFeb 6 2017, 4:37 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
config/
option/
12 lines

Diff 41646

View Options

src/applications/config/option/PhabricatorAuthenticationConfigOptions.php

Loading...
Phabricator · Built by Phacility