UI/Remove spinner after token renew (#12887)

* fixes loading spinner * adds changelog
hashicorp · Oct 21, 2021 · e8e1344 · e8e1344
1 parent ef43902
commit e8e1344
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 6 deletions.
diff --git a/changelog/12887.txt b/changelog/12887.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+ui: Remove spinner after token renew
+```
diff --git a/ui/app/components/auth-info.js b/ui/app/components/auth-info.js
@@ -40,8 +40,9 @@ export default class AuthInfoComponent extends Component {
   renewToken() {
     this.fakeRenew = true;
     run.later(() => {
-      this.fakeRenew = false;
-      this.auth.renew();
+      this.auth.renew().then(() => {
+        this.fakeRenew = this.auth.isRenewing;
+      });
     }, 200);
   }
 

diff --git a/ui/app/services/auth.js b/ui/app/services/auth.js
@@ -23,6 +23,7 @@ export default Service.extend({
   namespaceService: service('namespace'),
   IDLE_TIMEOUT: 3 * 60e3,
   expirationCalcTS: null,
+  isRenewing: false,
   init() {
     this._super(...arguments);
     this.checkForRootToken();
@@ -249,14 +250,14 @@ export default Service.extend({
     if (currentlyRenewing) {
       return;
     }
-    this.set('isRenewing', true);
+    this.isRenewing = true;
     return this.renewCurrentToken().then(
       resp => {
-        this.set('isRenewing', false);
+        this.isRenewing = false;
         return this.persistAuthData(tokenName, resp.data || resp.auth);
       },
       e => {
-        this.set('isRenewing', false);
+        this.isRenewing = false;
         throw e;
       }
     );

diff --git a/ui/app/templates/components/auth-info.hbs b/ui/app/templates/components/auth-info.hbs
@@ -26,7 +26,7 @@
         {{#if (is-before (now interval=1000) this.auth.tokenExpirationDate)}}
           {{#if this.auth.authData.renewable}}
             <li class="action">
-              <button type="button" {{action "renewToken"}} class="link button {{if this.isRenewing 'is-loading'}}">
+              <button type="button" {{on "click" this.renewToken}} class="link button {{if this.isRenewing 'is-loading'}}">
                 Renew token
               </button>
             </li>