Skip to content
This repository has been archived by the owner on Feb 7, 2019. It is now read-only.

Adding route definition to example #17

Closed
wants to merge 1 commit into from

Conversation

jskulski
Copy link

@jskulski jskulski commented Feb 4, 2016

This tripped me up a bit, so thought explicitly stating the order of decorators could help.

This works (404s):

@app.route('/feature-flag-test-route')
@is_active_feature('test_route')
def feature_flag_test_route():
    return 'on!'

This does not (returns 200, displays 'on!')

@is_active_feature('test_route')
@app.route('/feature-flag-test-route')
def feature_flag_test_route():
    return 'on!'

@jskulski
Copy link
Author

jskulski commented Mar 7, 2018

Closing for inactivity. Feel free to reopen :)

@jskulski jskulski closed this Mar 7, 2018
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

1 participant