-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
async / await debug in chrome #6056
Comments
Hey @jaynsw, in order to be able to help you with this issue you need to provide some code samples. What doesn't work exactly? Where did you put the breakpoint? I have a question for you. Did you see the New Issue Guidelines when creating this issue? Is there anything we could improve there? Is the text too long or unclear for example? |
I'll post my experience here, since the OP hasn't responded yet. Debugging async/await in chrome is extremely difficult. Maybe because of my inexperience with the inner workings of how it works, I'm not sure. But when you have code like so: 1. let response = await getSomethingFromTheNetwork();
2. let bar = response.foo;
3. doSomethingWithFoobar(bar); In Chrome if I place a breakpoint at line 2, the debugger will pause the application on that line and Does this make sense? |
I have changed back all of my code to promise then and catch.
|
Thanks for the example @Somojojojo! Any idea how to make Chrome show you the value of |
Besides using the console JS API I haven't found a solid way of getting those values. Again, as I have no idea how React is managing the debug session, my explanation likely sounds silly or obvious – but maybe it will give someone a good idea on how to solve this. |
This looks like a feature request and I'm not sure anyone on the team has bandwidth to look into it. Let's vote on it on Product Pains (voting is not possible on GitHub) so we see how high-priority this is. |
@facebook-github-bot feature |
Hey @jaynsw! Thanks for opening the issue, however it looks like a feature request. As noted in the Issue template we'd like to use the GitHub issues to track bugs only. Can you implement the feature as a standalone npm module? If not consider sending a pull request or a create an entry on Product Pains. It has a voting system and if the feature gets upvoted enough it might get implemented. |
Hey @jaynsw! Thanks for opening the issue, but we are closing it because it looks like a feature request. If this can't be implemented as a standalone npm module free to send us a Pull Request or create an entry on Product Pains. It has a voting system and if the feature gets upvoted enough it might get implemented.' |
Hi there! This issue is being closed because it has been inactive for a while. But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/async-await-debug-in-chrome ProductPains helps the community prioritize the most important issues thanks to its voting feature. Also, if this issue is a bug, please consider sending a pull request with a fix. |
@mkonicek Hello... has this feature been implemented? If not, is there a workaround? |
I believe async/await debugging in Chrome is currently broken: https://bugs.chromium.org/p/chromium/issues/detail?id=833928 @aslushnikov FYI |
@Somojojojo I know this thread is old but I am having a similar experience. Are you still using react native and seeing the same difficult async/await debugging experience in Chrome? If so, did you come up with a better workaround for debugging? |
I'm still experiencing that using RN 0.55.4 Are there any workarounds available? |
Ok it looks like the chromium bug is still there, so there isn't much we can do about it for now AFAIK. Still, let's reopen this. |
Since this is not actionable in the RN repo and an issue with Chromium, let's close this issue here since we can't do anything about it. |
after changing the promise to async await, I found debugging in chrome is impossible. Is there any plan to fix this in the near future or should I switch back to promise ?
The text was updated successfully, but these errors were encountered: