Ik admin add hall

The

```ruby
if hall_assignment.user.valid?
```

Seems weird. Primarily I don't like trailing conditionals, but why would you have hall_assignments for invalid users?