Page MenuHomekolab.org
Differential D2918

Add artisan owner:swap command
ClosedPublic
Actions

Authored by machniak on Oct 1 2021, 4:13 PM.

Details

Reviewers
mollekopf
Group Reviewers
Restricted Project
Commits
rK86ea94306654: Add artisan owner:swap command
Test Plan

./phpunit tests/Feature/Console/OwnerSwapTest.php

Diff Detail

Repository
rK kolab
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

machniak requested review of this revision.Oct 1 2021, 4:13 PM
machniak created this revision.
machniak updated this revision to Diff 8354.Oct 1 2021, 4:18 PM
  • Add todo comment
machniak updated this revision to Diff 8360.Oct 4 2021, 9:01 AM
  • Update mollie customer email/name on owner swap
mollekopf accepted this revision.Oct 6 2021, 9:51 AM
mollekopf added a subscriber: mollekopf.

Looks good.

Notes (no blockers I think):

  • The wallet description is not copied
  • It's not clear to me why the targetWallet needs to be one month old, perhaps add a comment.
  • The "same account" check is a bit obscure and would deserve a dedicated method I think: User::controlledBySameWallet($users)
This revision is now accepted and ready to land.Oct 6 2021, 9:51 AM
Closed by commit rK86ea94306654: Add artisan owner:swap command (authored by machniak). · Explain WhyOct 6 2021, 12:48 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 8381

View Options

src/app/Console/Commands/OwnerSwapCommand.php

Loading...
View Options

src/tests/Feature/Console/OwnerSwapTest.php

Loading...