Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Utilise permission_required pour la gestion des droits – module pages #4189

Merged
merged 1 commit into from
Feb 9, 2017
Merged

Utilise permission_required pour la gestion des droits – module pages #4189

merged 1 commit into from
Feb 9, 2017

Conversation

gllmc
Copy link
Member

@gllmc gllmc commented Feb 8, 2017

Q R
Type de modification refactorisation
Ticket(s) (issue(s)) concerné(s) #4181

Voici la PR utilisant permission_required pour la gestion des droits comme décrit dans #4181 pour le module pages (pour le coup, c'était le plus simple, il n'y avait qu'une vue :p).

QA

Code review uniquement.

@coveralls
Copy link

coveralls commented Feb 8, 2017

Coverage Status

Coverage increased (+0.01%) to 88.375% when pulling 8ee6c4c on GCodeur:decorateur_perms_pages into 5582cd3 on zestedesavoir:dev.

@coveralls
Copy link

Coverage Status

Coverage decreased (-33.9%) to 54.449% when pulling 8ee6c4c on GCodeur:decorateur_perms_pages into 5582cd3 on zestedesavoir:dev.

@artragis
Copy link
Member

artragis commented Feb 9, 2017

@pierre-24 this is mergeable.

@pierre-24
Copy link
Member

If you say so :)

@pierre-24 pierre-24 merged commit 2789cd1 into zestedesavoir:dev Feb 9, 2017
@vhf vhf added this to the Version de développement milestone Feb 10, 2017
@gllmc gllmc deleted the decorateur_perms_pages branch July 17, 2017 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants