Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

fix(loader): expose $$minErr to modules such asngResource #5050

Closed

Conversation

petebacondarwin
Copy link
Contributor

This is highlighted in angular-phonecat when you try to use the index-async.html
which needs to load the ngResource module asynchronously but fails when it tries
to call angular.$$minErr to create the $resourceMinErr object.

This will finally solve the issues blocking angular/angular-phonecat#90

This is highlighted in angular-phonecat when you try to use the index-async.html
which needs to load the ngResource module asynchronously but fails when it tries
to call `angular.$$minErr` to create the $resourceMinErr object.
IgorMinar pushed a commit to IgorMinar/angular.js that referenced this pull request Nov 22, 2013
This is highlighted in angular-phonecat when you try to use the index-async.html
which needs to load the ngResource module asynchronously but fails when it tries
to call `angular.$$minErr` to create the $resourceMinErr object.

Closes angular#5050
@IgorMinar
Copy link
Contributor

I made a small refactoring and landed this change.

@IgorMinar IgorMinar reopened this Nov 22, 2013
@IgorMinar IgorMinar closed this Nov 22, 2013
jamesdaily pushed a commit to jamesdaily/angular.js that referenced this pull request Jan 27, 2014
This is highlighted in angular-phonecat when you try to use the index-async.html
which needs to load the ngResource module asynchronously but fails when it tries
to call `angular.$$minErr` to create the $resourceMinErr object.

Closes angular#5050
jamesdaily pushed a commit to jamesdaily/angular.js that referenced this pull request Jan 27, 2014
This is highlighted in angular-phonecat when you try to use the index-async.html
which needs to load the ngResource module asynchronously but fails when it tries
to call `angular.$$minErr` to create the $resourceMinErr object.

Closes angular#5050
@petebacondarwin petebacondarwin deleted the loader-minErr branch February 11, 2014 09:56
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.

2 participants