Skip to content

Commit e7c0aa1

Browse files
committed
Add ng-strict-di option to ng-app directives
This isn't a feature in Angular 1.2, but when we upgrade to Angular 1.3, this will spew warnings for objects that rely on implicit injection behaviour (i.e. that are missing injection annotations) -- such objects are not suitable for minification.
1 parent d79a43b commit e7c0aa1

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

h/templates/app.html.jinja2

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,8 @@
77
{% endblock %}
88

99
{% block body_tag %}
10-
<body ng-app="h" ng-controller="AppController" ng-csp="">
10+
{# ng-strict-di is a (harmless) noop until we upgrade to Angular 1.3 #}
11+
<body ng-app="h" ng-strict-di ng-controller="AppController" ng-csp="">
1112
{% endblock %}
1213

1314
{% block content %}

h/templates/auth.html.jinja2

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,8 @@
77
{% endblock %}
88

99
{% block body_tag %}
10-
<body ng-app="h" ng-controller="AuthAppController">
10+
{# ng-strict-di is a (harmless) noop until we upgrade to Angular 1.3 #}
11+
<body ng-app="h" ng-strict-di ng-controller="AuthAppController">
1112
{% endblock %}
1213

1314
{% block content %}

0 commit comments

Comments
 (0)