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
Loading type=systemjs-module scripts. Resolves #2002. #2015
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice to see this.
@guybedford I just updated this with your feedback |
The only other thing I am wondering here is if we should do an immediate run of the function when SystemJS executes to ensure the request is sent out asap. System.import calls are self-deduping so there's no risk in double loading. I tend to think trying to give the best network perf is a good idea. Then we should also add some docs on this too. |
Ah yeah, good idea about immediately running it. I'll add that and also some docs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some final notes, then I think we're good to go.
Co-Authored-By: Guy Bedford <guybedford@gmail.com>
Co-Authored-By: Guy Bedford <guybedford@gmail.com>
Co-Authored-By: Guy Bedford <guybedford@gmail.com>
Updated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probable merge this into a dev / feature branch before release due to the docs change.
Merged in 6.x. |
See #2002