-
-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support initialization directly on Blueprint Group #182
Comments
Hello, can you give some suggestion how to implement this logic in the best way? |
Yes. 🤔 I would love to add this, so if you are interested in working on it, I am happy to help you. TBH, I have not put much thought into this lately. Iterating over each BP might work, but I would wonder what impact that would have. It does raise questions: should they have a shared config or not? where to store the |
I think, that, for first time, we can make blueprint group support available when only one App is used, otherwise it will be harder to implement and take more time for develop.. |
that seems a fair restriction. |
Just make sure that when someone tries to attach to a BP group, we check the Sanic registry size (it's a class variable) that we raise an appropriate exception. |
#181
The text was updated successfully, but these errors were encountered: