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

Unhandled errors on loading the app #112

Closed
vikasrohit opened this issue Jul 12, 2017 · 7 comments
Closed

Unhandled errors on loading the app #112

vikasrohit opened this issue Jul 12, 2017 · 7 comments
Assignees

Comments

@vikasrohit
Copy link

Since last few releases, we are observing multiple instance of following error in console. Though they does not seem to do any significant harm right now, it is better to handle it by identifying the root cause of it and fixing it.

app.c20966f….js:sourcemap:438 Error: transition prevented
    at L [as $get] (app.c20966f….js:sourcemap:23)
    at Object.a [as invoke] (app.c20966f….js:sourcemap:436)
    at app.c20966f….js:sourcemap:436
    at n (app.c20966f….js:sourcemap:436)
    at r (app.c20966f….js:sourcemap:436)
    at Object.a [as invoke] (app.c20966f….js:sourcemap:436)
    at app.c20966f….js:sourcemap:436
    at c (app.c20966f….js:sourcemap:436)
    at le (app.c20966f….js:sourcemap:436)
    at a (app.c20966f….js:sourcemap:436) "Possibly unhandled rejection: {}"
@vikasrohit vikasrohit self-assigned this Jul 12, 2017
@vikasrohit
Copy link
Author

@vikasrohit
Copy link
Author

The root cause is the angular release update starting 1.5.8. Seems like ui-router's code is causing these errors. More information can be found here.

Fix is to upgrade the ui-router dependency to 1.x release as suggested here.

@vikasrohit
Copy link
Author

PR #113 created for the same. @ykohata it would be great if you can review the changes.

@fnisen
Copy link

fnisen commented Aug 3, 2017

@vikasrohit looks like Kohata-san reviewed. Can we close this?

@vikasrohit
Copy link
Author

@fnisen With these changes, there is still some warnings when we install dependencies using npm. I am trying to get rid of them as well.

@vikasrohit
Copy link
Author

Done with those changes as well. It should be good now. I will merge it before we deploy accounts-app to prod.

@fnisen
Copy link

fnisen commented Aug 8, 2017

Beautiful, thanks!

@fnisen fnisen closed this as completed Aug 8, 2017
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

No branches or pull requests

2 participants