Here is a great example of what I'm talking about. Refactor this method:

```ruby
def action_allowed?
     if current_user.role.name.eql?("Instructor") || current_user.role.name.eql?("Teaching-Assistant") || current_user.role.name.eql?("Administrator")
       true
     else if current_user.role.name.eql?('Student')
            if action_name == 'add_dynamic_reviewer' || action_name =='release_reservation' || action_name == 'show_available_submissions' || action_name  == 'assign_reviewer_dynamically' || action_name  == 'assign_metareviewer_dynamically'
               true
            else
              false
            end
          end
     end
   end
```

To this:

```ruby
def action_allowed?
  ["Instructor",
  "Teaching-Assistant",
  "Administrator",
  "Student"].include?(current_user.role.name)) &&
  ["add_dynamic_reviewer",
  "release_reservation",
  "show_available_submissions",
  "assign_reviewer_dynamically"].include?(action_name)
end
```

This is _much_ easier to read, and it doesn't go over the 80 character width limit.