Replies: 3 comments 2 replies
-
Oh, I have to admit I didn’t know about app.extension! |
Beta Was this translation helpful? Give feedback.
0 replies
-
Looks like it doesn’t link back, so here’s the issue: #72 |
Beta Was this translation helpful? Give feedback.
2 replies
-
fixed by #73 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
This is a minor nit, but I find that
app.config
is usually filled with things which might be set from an externalconfig.py
file, environment variable, or some other source. I often even like to be able to render these things back to e.g. JSON to save the current configuration for debugging.OTOH, flask provides
app.extensions
as a specific place for extensions to store state, and many flask extensions do that (e.g. Flask-SQLAlchemy).What do you think about having
svcs
store theRegistry
object (which is managed bysvcs
, and wouldn't make much sense to set as the end user) inapp.extensions
?Beta Was this translation helpful? Give feedback.
All reactions