Differential D26

New artisan command: account:reset
ClosedPublic
Actions

Authored by dereckson on Jul 13 2015, 21:12.

Details

Reviewers

dereckson

Maniphest Tasks

T482: [CLI] Reset password command

Commits

rGROVEe3d721c5d7e9: New artisan command: account:reset

Summary

./artisan account:reset <id, username or mail>

Fixes T482.

Test Plan

./artisan account:reset 5
./artisan account:reset quux --subject="Quuxians, révoltez-vous"

Diff Detail

Repository

rGROVE Auth Grove

Lint

No Lint Coverage

Unit

No Test Coverage

Branch

artisan-account-reset

Build Status

Buildable 11
Build 11: arc lint + arc unit

Event Timeline

dereckson updated this revision to Diff 52.Jul 13 2015, 21:12

dereckson retitled this revision from to New artisan command: account:reset.

dereckson updated this object.

dereckson edited the test plan for this revision. (Show Details)

dereckson added a reviewer: dereckson.

dereckson added a task: T482: [CLI] Reset password command.

dereckson planned changes to this revision.Jul 13 2015, 21:24

dereckson added inline comments.

app/Console/Commands/AccountReset.php
35	Whitespace issue.
48	Regardless of the misspelling, l10n is the preferred form.
72	Whitespace issue.
78	Whitespace issue.
90	Whitespace issue.

Two other issues:

app/Console/Commands/AccountReset.php
9	Extraneous \n
12	No need to rename this one.

Whitespace fixes, l10n.

Looks good.

dereckson accepted this revision.Jul 13 2015, 21:36

dereckson edited edge metadata.

This revision is now accepted and ready to land.Jul 13 2015, 21:36

Closed by commit rGROVEe3d721c5d7e9: New artisan command: account:reset (authored by dereckson, committed by dereckson). · Explain WhyJul 13 2015, 21:37

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

app/

Console/

Commands/

AccountReset.php

130 lines

Kernel.php

1 line

Diff 53

View Options

app/Console/Commands/AccountReset.php