LoCo Team Portal

Code review comment for lp:~cjohnston/loco-team-portal/bugfix-lp-610040

  1. bugfix-lp-610040
  2. Merge into 0.2
Revision history for this message
Daniel Holbach (dholbach) wrote :

 - is_admin_or_owner needs the user.is_authenticated() check too
 - "request.user.is_authenticated()" check in global_event_detail in events/views.py can be removed
 - I don't think we need DEBUG_USER in settings.py
 - can we make it DEBUG_USERS instead? so it'd be possible to specify more than one person?

Apart from that: good work.

review: Needs Fixing

« Back to merge proposal