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

[stable7] Always add to $loadedApps #18845

Merged
merged 1 commit into from
Sep 6, 2015

Conversation

RobinMcCorkell
Copy link
Member

Fixes the RouteNotFoundException on app upgrade issue.

Partial backport of #18839

cc @karlitschek @LukasReschke @MorrisJobke @DeepDiver1975

I don't know if we want to backport this all the way to stable7, but it's such a small change I thought why not.

@scrutinizer-notifier
Copy link

The inspection completed: No new issues

@karlitschek
Copy link
Contributor

👍

@MorrisJobke MorrisJobke added this to the 7.0.10-current-maintenance milestone Sep 5, 2015
@MorrisJobke
Copy link
Contributor

Tested and works 👍

MorrisJobke added a commit that referenced this pull request Sep 6, 2015
…ble7

[stable7] Always add to $loadedApps
@MorrisJobke MorrisJobke merged commit cde3efb into stable7 Sep 6, 2015
@MorrisJobke MorrisJobke deleted the app-upgrade-routenotfound-stable7 branch September 6, 2015 22:24
@lock lock bot locked as resolved and limited conversation to collaborators Aug 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants