-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Cannot start a nested transaction #15255
Comments
Can someone please explain the meaning of this assertion? Against what type of error does it guard? A usage problem to fix in the app or a glimmer internal problem? |
I am encountering this issue as well. Only when using |
Same issue here. Slightly different symptoms.
|
When I have ran into this, it has been a red herring of sorts. Basically some other error occurs during render, putting the VM into a bad state, which results in this error. So although glimmer also needs to address being able to recover correctly, you may want to look for other errors in the console and address them. (my example was someone trying to use |
hypothetical fix: glimmerjs/glimmer-vm#505 (i need to actually write tests and confirm, so don't get too exited yet) |
is there some news about this issue? is not, could you please guys provide some workaround? it blocks our company migration from 1.13.12 to 2.13.2 |
2.13.3 was released this morning with a fix for this assertion.
On Wed, May 31, 2017 at 12:43 PM Alexander Kapelyukhovskiy < ***@***.***> wrote:
is there some news about this issue? is not, could you please guys provide
some workaround? it blocks our company migration from 1.13.12 to 2.13.2
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#15255 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABQ5qkf9uMzPMicJsnro5IBC5_EmHbEMks5r_cLigaJpZM4Ngrh6>
.
--
Jordan Hawker
www.JordanHawker.com
Senior JavaScript Engineer, Twitch
|
@elwayman02 thanks! I forgot to close. |
Is 2.13.3 going to be released on NPM? |
Yep, it was released to npm at the same time as it was to bower (roughly 21 hours ago). |
@rwjblue, could you please clarify which packages I need to update to fix |
@AlexanderKapelyukhovskiy ember-source should be at-least 2.13.3 i believe |
Confirm |
As I didn't have ember-source in npm dependencies I've updated bower ember package to 2.13.3 and looks like it helped with Only one question I have - was this issue due to usage of bower package and if not how can I fix it if I remove bower from our project. |
@AlexanderKapelyukhovskiy I upgraded some time ago like you did and I noticed that all of the bower packages were moved to npm. You can get a list of changes of your bower.json and package.json like this: It looks like there is no ember-new-output for the v2.13.3 yet, but basically it's the same |
@RobIsHere / @AlexanderKapelyukhovskiy ah i forgot about bower, but the 2.13.3 version of ember in bower should also resolve the issue. |
Upgrading to 2.13.1 and see the following errors,
I suppose this is related to the back tracking issues but the error message makes it hard to work back towards the cause. Any ideas on how we would go about debugging this?
The text was updated successfully, but these errors were encountered: