Added artisan user:restore and domain:restore commands
Details
Details
Diff Detail
Diff Detail
- Repository
- rK kolab
- Branch
- dev/user-restore
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 32533 Build 12556: arc lint + arc unit
Event Timeline
src/app/Domain.php | ||
---|---|---|
271 ↗ | (On Diff #5308) | Isn't STATUS_LDAP_READY removed when the associated job is done? |
src/app/Jobs/Domain/CreateJob.php | ||
20 ↗ | (On Diff #5308) | Marking a domain as deleted in LDAP is still valid, though, so I'd expect the domain to show up in LDAP with inetdomainstatus &= 8. |
src/app/Observers/DomainObserver.php | ||
134 | When a domain is restored, we don't update the updated_at column of the entitlements table because it's used to track payment status / due dates for new charges. | |
150 | I believe created_at will remain what it was, which would be the expected result. I also believe $domain->save() will automatically update updated_at. | |
src/app/Observers/UserObserver.php | ||
290 | Here too, please avoid touching updated_at. | |
src/app/User.php | ||
733 ↗ | (On Diff #5308) | This should be an effect of having the associated jobs executed successfully, no? |
Comment Actions
- Complete restoring/restored events handlers in user and domain observers
- Add test for user:restore command
- Add domain:restore command
Comment Actions
- Complete restoring/restored events handlers in user and domain observers
- Add test for user:restore command
- Add domain:restore command
- Set proper status in user/domain DeleteJob