Summary

Show the standard user selector, not an ID list, for a group's users.

Review Request #8130 — Created April 27, 2016 and submitted May 17, 2016, 1:38 p.m.

Information

Owner

chipx86

Repository

Review Board

Branch

release-2.5.x

Bugs

Depends On

Commit

8a25271...

Reviewers

Groups

reviewboard

People

Description

A previous change regressed the usablity of the review group page,
swapping the user selector for an ID list for performance reasons. This
is really only an issue on our spam-filled server, which is a whole
separate issue, and we want admins to not have to deal with matching up
IDs.

This switches us back to the user selector.

Testing Done

Verified that the user selector was back.

Tool: Pyflakes
Processed Files:
    reviewboard/reviews/admin.py



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/reviews/admin.py

I don't think it's just our "spam-filled server". There are other public servers with very many legitimate users who had apache thread crashing issues because of this. I think we need to write our own widget (or find some third-party one) that combines the search functionality of the ID field with the visual UI of the selector.

chipx86 April 27, 2016, 2:02 p.m.

I think we should too, but this is a really terrible experience by default, and it'd be enough to make me look at alternatives. It's basically unusable for the entire purpose of that page. Until we have such a selector, the behavior we had for years is going to be a better experience for 99% of the installs.

chipx86 April 27, 2016, 2:04 p.m.

Want to try to get a new widget written before the next 2.5.x? That'd definitely be an ideal scenario.

david April 27, 2016, 2:07 p.m.

I'll see what I can do. It's definitely a rock/hard-place situation because either we have a bad UI or we crash for the people who rely on it most.

chipx86 May 16, 2016, 9:20 p.m.

So the new widget is in development, and we'll hopefully be able to get that into a release soon, but I'd like to get this change into 2.5.5 (tomorrow) until then.

Companies with the large servers can stay on 2.5.4 if they need to, but having first-hand gone through the setup process with this change in place, it basically makes our product unusabe from a setup and maintenance standpoint. Plus it doesn't solve the problem, because the repository and default reviewer pages still show the original widget.

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to release-2.5.x (fc63fae)