Page MenuHomePhabricator
Differential D20007

Update documentation to reflect bin/auth changes
ClosedPublic
Actions

Authored by amckinley on Jan 21 2019, 7:59 PM.
Tags
None
Referenced Files
F14081086: D20007.diff
Fri, Nov 22, 6:23 PM
Unknown Object (File)
Tue, Nov 19, 6:03 PM
Unknown Object (File)
Fri, Nov 15, 4:46 PM
Unknown Object (File)
Sun, Nov 10, 6:04 PM
Unknown Object (File)
Wed, Nov 6, 1:37 AM
Unknown Object (File)
Oct 21 2024, 4:26 AM
Unknown Object (File)
Oct 16 2024, 8:17 AM
Unknown Object (File)
Oct 15 2024, 5:17 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rP6138d5885d23: Update documentation to reflect bin/auth changes
Summary

See https://secure.phabricator.com/D18901#249481. Update the docs and a warning string to reflect the new reality that bin/auth recover is now able to recover any account, not just administrators.

Test Plan

Mk 1 eyeball

Diff Detail

Repository
rP Phabricator
Branch
authrecover (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21623
Build 29478: Run Core Tests
Build 29477: arc lint + arc unit

Event Timeline

amckinley created this revision.Jan 21 2019, 7:59 PM
amckinley mentioned this in D18901: Remove "set password" from `bin/accountadmin` and let `bin/auth recover` recover anyone.Jan 21 2019, 8:00 PM
Harbormaster completed remote builds in B21623: Diff 47767.Jan 21 2019, 8:01 PM
amckinley requested review of this revision.Jan 21 2019, 8:01 PM
Herald added a subscriber: Korvin. · View Herald TranscriptJan 21 2019, 8:01 PM
epriestley accepted this revision.Jan 21 2019, 8:02 PM
epriestley added inline comments.
src/docs/user/configuration/configuring_accounts_and_registration.diviner
31

Maybe "Recovering Inaccessible Accounts" or similar, without context I think of "locked" as "someone explicitly locked it" (disable/banish/etc).

This revision is now accepted and ready to land.Jan 21 2019, 8:02 PM
amckinley updated this revision to Diff 47768.Jan 21 2019, 8:10 PM

Wordsmithing.

Harbormaster completed remote builds in B21624: Diff 47768.Jan 21 2019, 8:13 PM
Closed by commit rP6138d5885d23: Update documentation to reflect bin/auth changes (authored by amckinley). · Explain WhyJan 21 2019, 8:20 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
auth/
controller/
2 lines
docs/
user/
configuration/
17 lines

Diff 47767

View Options

src/applications/auth/controller/PhabricatorAuthStartController.php

Loading...
View Options

src/docs/user/configuration/configuring_accounts_and_registration.diviner

Loading...
Phabricator · Built by Phacility