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

Review Request #8130 — Created April 27, 2016 and submitted

Information

Review Board
release-2.5.x
8a25271...

Reviewers

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.

Verified that the user selector was back.

reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/reviews/admin.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/reviews/admin.py
    
    
  2. 
      
david
  1. 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.

    1. 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.

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

    3. 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.

    4. 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.

  2. 
      
david
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to release-2.5.x (fc63fae)