Page MenuHomePhorge

Display users with no ACTIVE flag as Not Ready, code cleanup
ClosedPublic

Authored by machniak on Mar 27 2023, 1:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 25, 8:50 AM
Unknown Object (File)
Fri, Nov 24, 2:55 AM
Unknown Object (File)
Fri, Nov 17, 7:24 AM
Unknown Object (File)
Tue, Nov 14, 5:52 PM
Unknown Object (File)
Sun, Nov 12, 9:13 AM
Unknown Object (File)
Wed, Nov 8, 6:56 PM
Unknown Object (File)
Wed, Nov 8, 6:55 PM
Unknown Object (File)
Wed, Nov 8, 6:54 PM
Subscribers
Restricted Project

Details

Test Plan

./phpunit

Diff Detail

Repository
rK kolab
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

machniak created this revision.
mollekopf added inline comments.
src/app/Observers/DomainObserver.php
95

So should we perhaps just do a $domain->status = Domain::STATUS_NEW? Or what do we intend to preserve?

src/app/Observers/DomainObserver.php
95

I suppose the initial idea was to preserve CONFIRMED and VERIFIED state, but now I think every restored domain should get NEW state only. This is not a show stopper for this diff.

  • Make domain NEW (and only NEW) on restore
This revision is now accepted and ready to land.Mar 30 2023, 5:55 PM
This revision was automatically updated to reflect the committed changes.