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

fix(typescript-webpack-experimental): Replace executeAyncScript with executeScript #4

Merged
merged 1 commit into from
Feb 5, 2017
Merged

fix(typescript-webpack-experimental): Replace executeAyncScript with executeScript #4

merged 1 commit into from
Feb 5, 2017

Conversation

michaelbull
Copy link
Contributor

@michaelbull michaelbull commented Feb 5, 2017

Fixes the issue described in: aurelia/skeleton-navigation#714 (comment)

Previously when the app was larger it would wait to emit the events before
protractor was loaded, however due to the speed increase introduced by
the experimental webpack configuration, aurelia was initialising and emitting
the event before protractor had a chance to listen to it, hence it would hang
indefinitely. Telling the browser to execute the script synchronously will
delay aurelia long enough for protractor to register its listeners before
aurelia emits the event itself.

This solution was provided by @niieani.

…executeScript

Fixes the issue described in: aurelia/skeleton-navigation#714 (comment)

Previously when the app was larger it would wait to emit the events before
protractor was loaded, however due to the speed increase introduced by
the experimental webpack configuration, aurelia was initialising and emitting
the event before protractor had a chance to listen to it, hence it would hang
indefinitely. Telling the browser to execute the script synchronously will
delay aurelia long enough for protractor to register its listeners before
aurelia emits the event itself.
@CLAassistant
Copy link

CLAassistant commented Feb 5, 2017

CLA assistant check
All committers have signed the CLA.

@niieani
Copy link
Contributor

niieani commented Feb 5, 2017

Thank you.

@michaelbull michaelbull deleted the fix-async-script-execution branch February 5, 2017 18:16
@dweber019
Copy link

dweber019 commented Apr 18, 2017

@niieani Please release this. I already have the issue in my project with just a few more components ;)

@niieani
Copy link
Contributor

niieani commented Apr 18, 2017

Oh, didn't realize it wasn't! I've CC'd @EisenbergEffect about releasing.

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

Successfully merging this pull request may close these issues.

4 participants