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

WIP: Refactoring - cleaning up async/awaits #15707

Closed
wants to merge 1 commit into from

Conversation

nyurik
Copy link
Contributor

@nyurik nyurik commented Dec 20, 2017

No description provided.

@nyurik nyurik added the WIP Work in progress label Dec 20, 2017
@trevan
Copy link
Contributor

trevan commented Dec 20, 2017

I was under the impression that async/await couldn't be used in angular code. Did I miss some sort of discussion that resolved that issue?

@nyurik
Copy link
Contributor Author

nyurik commented Dec 20, 2017

@trevan thx for the heads up, i wasn't aware of that issue. Do you know the exact limitations, and whom to talk about it?

@trevan
Copy link
Contributor

trevan commented Dec 20, 2017

@nyurik, the original issue with async/await in angular is that it bypasses angular's digest loop and so changes done will not be caught by angular. I saw discussions about working around that (especially for shared code between angular and react). I did a quick search and found #13855 which has more discussion. I don't see any resolution there, though.

@nyurik
Copy link
Contributor Author

nyurik commented Jan 10, 2018

thanks for explaining. This should probably be shelved until better time.

@nyurik nyurik closed this Jan 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants