Add a decorators attribute to FlaskView #21

Merged
merged 1 commit into from Apr 8, 2013

Conversation

Projects
None yet
3 participants
@julienr

julienr commented Mar 26, 2013

This add a decorators attribute to FlaskView, letting user specify a list of decorators to be applied on all views.
This is similar to the decorators attribute on Flask's View and MethodView.

@apiguy

This comment has been minimized.

Show comment
Hide comment
@apiguy

apiguy Mar 28, 2013

This is a fantastic idea - just have to decide if I like the attribute to be just "decorators" or if something more explicit is in order.

apiguy commented on 72d875a Mar 28, 2013

This is a fantastic idea - just have to decide if I like the attribute to be just "decorators" or if something more explicit is in order.

@charlax

This comment has been minimized.

Show comment
Hide comment
@charlax

charlax Apr 3, 2013

👍

decorators seems like the best option to keep it consistent with Flask.

charlax commented Apr 3, 2013

👍

decorators seems like the best option to keep it consistent with Flask.

apiguy added a commit that referenced this pull request Apr 8, 2013

Merge pull request #21 from julienr/decorators
Add a decorators attribute to FlaskView

@apiguy apiguy merged commit f02a8f4 into apiguy:master Apr 8, 2013

@apiguy

This comment has been minimized.

Show comment
Hide comment
@apiguy

apiguy Apr 8, 2013

Owner

Thanks @julienr I'm going to play with this a bit more but it'll be in the next version! Awesome idea.

Owner

apiguy commented Apr 8, 2013

Thanks @julienr I'm going to play with this a bit more but it'll be in the next version! Awesome idea.

@julienr

This comment has been minimized.

Show comment
Hide comment
@julienr

julienr Apr 8, 2013

Great, thanks !

julienr commented Apr 8, 2013

Great, thanks !

@julienr julienr deleted the julienr:decorators branch Apr 8, 2013

fbarresi pushed a commit to fbarresi/flask-classy that referenced this pull request Jan 14, 2017

fbarresi pushed a commit to fbarresi/flask-classy that referenced this pull request Jan 14, 2017

fbarresi pushed a commit to fbarresi/flask-classy that referenced this pull request Jan 14, 2017

fbarresi pushed a commit to fbarresi/flask-classy that referenced this pull request Jan 14, 2017

fbarresi pushed a commit to fbarresi/flask-classy that referenced this pull request Jan 14, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment