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
F15422561: D20007.id47768.diff
Sat, Mar 22, 9:22 AM
F15421525: D20007.id47771.diff
Sat, Mar 22, 12:35 AM
F15417308: D20007.diff
Thu, Mar 20, 4:35 PM
F15282941: D20007.diff
Tue, Mar 4, 3:48 AM
Unknown Object (File)
Feb 18 2025, 11:16 AM
Unknown Object (File)
Feb 9 2025, 6:58 AM
Unknown Object (File)
Feb 9 2025, 6:58 AM
Unknown Object (File)
Feb 9 2025, 6:58 AM
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