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
-
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