Skip to content
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

module-level context #70

Open
vbelolapotkov opened this issue Feb 8, 2016 · 1 comment
Open

module-level context #70

vbelolapotkov opened this issue Feb 8, 2016 · 1 comment

Comments

@vbelolapotkov
Copy link
Contributor

As per spec modules are not allowed to update application context. So module heavily depends on context. At least it uses context to connect actions and containers together.

On the other hand context is the app specific thing and may not contain all objects used by module. So to reuse modules we have to preconfigure context for each module. And this can lead us to conflicts when the same context name required by different modules for different purposes.

To break this dependency It would be great to have the option to store in the "module context" value returned by load function.
This will eliminate the issue with naming conflicts and there will be no need to preconfigure app context for module.

I'd do this only for module specific storage/objects but not for meteor packages as it seems to be ok to have them in context.

Any thoughts on this?

@TeemoWan
Copy link

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants