From 37526aca880254d4200e175d69bba9bdfade0ba2 Mon Sep 17 00:00:00 2001
From: Paulus Schoutsen <paulus@paulusschoutsen.nl>
Date: Thu, 11 May 2017 20:45:17 -0700
Subject: [PATCH] Show validation errors in the UI when setting addon options

https://www.pivotaltracker.com/story/show/145323139
---
 .../addon-view/hassio-addon-options.html      | 30 +++++++++++++++----
 1 file changed, 25 insertions(+), 5 deletions(-)

diff --git a/panels/hassio/addon-view/hassio-addon-options.html b/panels/hassio/addon-view/hassio-addon-options.html
index cef2eac550..d8e12ddbb1 100644
--- a/panels/hassio/addon-view/hassio-addon-options.html
+++ b/panels/hassio/addon-view/hassio-addon-options.html
@@ -13,21 +13,26 @@
       paper-card {
         display: block;
       }
+      .errors {
+        color: var(--google-red-500);
+        margin-bottom: 16px;
+      }
       iron-autogrow-textarea {
         width: 100%;
       }
     </style>
     <paper-card heading='Options'>
       <div class="card-content">
+        <template is='dom-if' if='[[errors]]'>
+          <div class='errors'>[[errors]]</div>
+        </template>
         <iron-autogrow-textarea value="{{options}}"></iron-autogrow-textarea>
       </div>
       <div class="card-actions">
-        <ha-call-api-button
-          hass='[[hass]]'
+        <paper-button
+          on-tap='saveTapped'
           disabled='[[!optionsParsed]]'
-          data='[[computeOptionsData(optionsParsed)]]'
-          path="[[pathOptions(addon)]]"
-        >Save</ha-call-api-button>
+        >Save</paper-button>
       </div>
     </paper-card>
   </template>
@@ -56,6 +61,11 @@ Polymer({
       computed: 'parseOptions(options)',
     },
 
+    errors: {
+      type: String,
+      value: null,
+    },
+
     addonState: {
       type: Object,
       value: null,
@@ -75,6 +85,16 @@ Polymer({
     }
   },
 
+  saveTapped: function () {
+    this.errors = null;
+
+    this.hass.callApi('post', 'hassio/addons/' + this.addon + '/options', {
+      options: this.optionsParsed
+    }).catch(function (resp) {
+      this.errors = resp.body.message;
+    }.bind(this));
+  },
+
   computeOptionsData: function (optionsParsed) {
     return {
       options: optionsParsed,