I think this would be much easier to read if it were written as a sequence of condition statements, e.g.:
```python
def get_user_permissions(org, org_user):
    if not org_user.user.is_active:
        return 'inactive'
    elif org.is_owner(org_user.user):
        return 'owner'
    elif org_user.is_admin:
        return 'admin'
    elif org_user.user.has_perm('edit_decisions_feedback', org):
        return 'editor'
    else:
        return 'observer'
```

Apart from anything else, the code as it stands relies on the boolean expressions producing 0 or True, whereas there's nothing in particular to stop them producing any other "true" Python value.