Skip to content
This repository has been archived by the owner on Sep 26, 2022. It is now read-only.

Improve notify support #73

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Improve notify support #73

wants to merge 2 commits into from

Conversation

falood
Copy link

@falood falood commented Feb 26, 2014

with an ugly look....

@@ -68,7 +68,8 @@ def rendering_time(response):


def register_routes(app):
from .handlers import front, account, node, topic, user, admin
from .handlers import front, account, node, topic, user, admin, notify
app.register_blueprint(notify.bp, url_prefix='/notify')
app.register_blueprint(account.bp, url_prefix='/account')
app.register_blueprint(node.bp, url_prefix='/node')
app.register_blueprint(topic.bp, url_prefix='/topic')

Choose a reason for hiding this comment

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

Is return app at the end of func register_routes necessary?

Copy link
Contributor

Choose a reason for hiding this comment

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

no

Choose a reason for hiding this comment

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

so why not remove

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.

4 participants