Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

Reviewers #51

Merged
merged 13 commits into from Sep 26, 2014
Merged

Reviewers #51

merged 13 commits into from Sep 26, 2014

Conversation

pypingou
Copy link
Member

No description provided.

@@ -201,7 +218,8 @@ def decorated_function(*args, **kwargs):
flask.flash(
'You must be in one more group than the CLA', 'error')
return flask.redirect(flask.url_for('index'))
elif not is_nuancier_admin(flask.g.fas_user):
elif not is_nuancier_admin(flask.g.fas_user) \
and not is_nuancier_reviewer(flask.g.fas_user):
flask.flash('You are not an administrator of nuancier',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This error message is not quite accurate anymore..

@ralphbean
Copy link
Contributor

I left two comments to just clean stuff up at the edges. Besides those, it looks good. :)

@ralphbean
Copy link
Contributor

Cool -- 👍

pypingou added a commit that referenced this pull request Sep 26, 2014
@pypingou pypingou merged commit cad6eef into master Sep 26, 2014
@pypingou pypingou deleted the reviewers branch September 26, 2014 04:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants