Merge pull request #464 from mulesoft/qa

Render console if there are only warnings
mulesoft · Nov 24, 2016 · 6a284ef · 6a284ef
2 parents da6efaf + fd23904
commit 6a284ef
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 10 deletions.
diff --git a/README.md b/README.md
@@ -16,7 +16,7 @@
 
 ## API Designer online
 
-There is an online preview version of the API Designer, open the last stable version [here](http://mulesoft.github.io/api-designer/dists/master/latest/) or check out a [different branch](http://mulesoft.github.io/api-designer/).
+There is an online preview version of the API Designer, check it out a [different branch](http://mulesoft.github.io/api-designer/).
 
 ## Running Locally
 

diff --git a/dist/scripts/api-designer-vendor.js b/dist/scripts/api-designer-vendor.js
@@ -82376,10 +82376,18 @@ exports.javascript = require('./javascript');
         } else {
           return ramlParser.loadPath($window.resolveUrl(url), null, $scope.options)
             .then(function (api) {
-              if (api.errors.length <= 0) {
+              var success = true;
+              var issues = api.errors; // errors and warnings
+              if (issues && issues.length > 0) {
+                success = issues.filter(function (issue) {
+                    return !issue.isWarning;
+                  }).length === 0;
+              }
+
+              if (success) {
                 $scope.vm.raml = api.specification;
               } else {
-                $scope.vm.error = { message: 'Api contains errors.', errors : api.errors};
+                $scope.vm.error = { message: 'Api contains errors.', errors : issues};
               }
             })
             .finally(function () {
@@ -82875,11 +82883,19 @@ exports.javascript = require('./javascript');
 
         return promise
           .then(function (api) {
-            if (api.errors.length <= 0) {
+            var success = true;
+            var issues = api.errors; // errors and warnings
+            if (issues && issues.length > 0) {
+              success = issues.filter(function (issue) {
+                  return !issue.isWarning;
+                }).length === 0;
+            }
+
+            if (success) {
               $scope.vm.raml = api.specification;
             } else {
               $scope.vm.error           = { message: 'Api contains errors.'};
-              $scope.vm.codeMirror.lint = lintFromError(api.errors);
+              $scope.vm.codeMirror.lint = lintFromError(issues);
             }
           })
           .finally(function () {
@@ -88310,7 +88326,7 @@ angular.module('ramlConsoleApp').run(['$templateCache', function($templateCache)
     "            <span>{{ vm.error.message }}</span>\n" +
     "          </div>\n" +
     "          <div class=\"raml-console-error-pre\" ng-repeat=\"err in vm.error.errors\">\n" +
-    "            {{err.message}}\n" +
+    "            [{{err.isWarning && 'warning' || 'error'}}] {{err.message}}\n" +
     "          </div>\n" +
     "        </div>\n" +
     "      </section>\n" +