Page MenuHomePhorge

User RESTRICTED status
ClosedPublic

Authored by machniak on Feb 3 2023, 11:20 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 17, 7:58 AM
Unknown Object (File)
Fri, Oct 4, 2:29 PM
Unknown Object (File)
Thu, Oct 3, 12:52 PM
Unknown Object (File)
Sep 16 2024, 5:16 AM
Unknown Object (File)
Sep 13 2024, 8:37 AM
Unknown Object (File)
Sep 13 2024, 8:37 AM
Unknown Object (File)
Sep 8 2024, 4:45 AM
Unknown Object (File)
Sep 3 2024, 11:15 PM
Subscribers
Restricted Project

Details

Reviewers
mollekopf
Group Reviewers
Restricted Project
Commits
rK43d8fa9d3c2a: User RESTRICTED status
Summary

Restricted flag for new users
Fix the user:resync test that was removing existing users
Display Restricted badge on user page for admins
Update Dusk, fix flaky test

Test Plan

./phpunit

Diff Detail

Repository
rK kolab
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

machniak created this revision.
  • Small improvement, removed debug code
mollekopf added inline comments.
src/app/Http/Controllers/API/SignupController.php
318

Why is this change required? Is it just unrelated cleanup?

src/app/Http/Controllers/API/SignupController.php
318

Yes, it's unrelated cleanup. That's the only place in the controllers where we set NEW status, so for consistency I removed it. The NEW status is being set by observers.

This revision is now accepted and ready to land.Feb 3 2023, 12:53 PM
This revision was automatically updated to reflect the committed changes.