Update to ember 3.28 and address deprecations

Author: lukemelia

.eslintignore

@@ -13,6 +13,7 @@
 # misc
 /coverage/
 !.*
+.*/
 .eslintcache
 
 # ember-try

.eslintrc.js

@@ -24,21 +24,15 @@ module.exports = {
     // node files
     {
       files: [
-        '.eslintrc.js',
-        '.prettierrc.js',
-        '.template-lintrc.js',
-        'ember-cli-build.js',
-        'index.js',
-        'testem.js',
-        'blueprints/*/index.js',
-        'config/**/*.js',
-        'tests/dummy/config/**/*.js',
-      ],
-      excludedFiles: [
-        'addon/**',
-        'addon-test-support/**',
-        'app/**',
-        'tests/dummy/app/**',
+        './.eslintrc.js',
+        './.prettierrc.js',
+        './.template-lintrc.js',
+        './ember-cli-build.js',
+        './index.js',
+        './testem.js',
+        './blueprints/*/index.js',
+        './config/**/*.js',
+        './tests/dummy/config/**/*.js',
       ],
       parserOptions: {
         sourceType: 'script',
@@ -50,5 +44,10 @@ module.exports = {
       plugins: ['node'],
       extends: ['plugin:node/recommended'],
     },
+    {
+      // Test files:
+      files: ['tests/**/*-test.{js,ts}'],
+      extends: ['plugin:qunit/recommended'],
+    },
   ],
 };

.npmignore

@@ -15,6 +15,8 @@
 /.eslintrc.js
 /.git/
 /.gitignore
+/.prettierignore
+/.prettierrc.js
 /.template-lintrc.js
 /.travis.yml
 /.watchmanconfig
@@ -24,6 +26,7 @@
 /ember-cli-build.js
 /testem.js
 /tests/
+/yarn-error.log
 /yarn.lock
 .gitkeep
 

.template-lintrc.js

@@ -1,5 +1,5 @@
 'use strict';
 
 module.exports = {
-  extends: 'octane',
+  extends: 'recommended',
 };

CONTRIBUTING.md

@@ -8,9 +8,8 @@
 
 ## Linting
 
-* `npm run lint:hbs`
-* `npm run lint:js`
-* `npm run lint:js -- --fix`
+* `yarn lint`
+* `yarn lint:fix`
 
 ## Running tests
 

README.md

@@ -7,9 +7,9 @@ This addon's current status is "opensource, but unpolished". It lacks tests, doc
 Compatibility
 ------------------------------------------------------------------------------
 
-* Ember.js v3.16 or above
-* Ember CLI v2.13 or above
-* Node.js v10 or above
+* Ember.js v3.24 or above
+* Ember CLI v3.24 or above
+* Node.js v12 or above
 
 
 Installation

addon/components/nav-stack/component.js

@@ -5,8 +5,6 @@ import { run, scheduleOnce } from '@ember/runloop';
 import { nextTick } from 'ember-nav-stack/utils/animation';
 import BackSwipeRecognizer from 'ember-nav-stack/utils/back-swipe-recognizer';
 import Hammer from 'hammerjs';
-// import { argument } from '@ember-decorators/argument';
-// import { Action, optional } from '@ember-decorators/argument/types';
 import { Spring } from 'wobble';
 import { getOwner } from '@ember/application';
 import { DEBUG } from '@glimmer/env';

addon/components/nav-stack/template.hbs

@@ -22,11 +22,11 @@
   <div class="NavStack-header">
     {{#if this.parentItemHeaderComponent}}
       <div class="NavStack-headerContainer NavStack-parentItemHeaderContainer">
-        {{component this.parentItemHeaderComponent class="NavStack-headerComponent"}}
+        {{component this.parentItemHeaderComponent class="NavStack-headerComponent" back=@back}}
       </div>
     {{/if}}
     <div class="NavStack-headerContainer NavStack-currentHeaderContainer">
-      {{component this.headerComponent class="NavStack-headerComponent"}}
+      {{component this.headerComponent class="NavStack-headerComponent" back=@back}}
     </div>
   </div>
   {{#if this.hasFooter}}

addon/mixins/stackable-route.js renamed to addon/routes/stackable-route.js

@@ -1,6 +1,6 @@
-/* eslint-disable ember/no-new-mixins */
-import Mixin from '@ember/object/mixin';
-import { computed } from '@ember/object';
+import Route from '@ember/routing/route';
+import { action } from '@ember/object';
+import { inject as service } from '@ember/service';
 
 export function getParentRoute(router, route) {
   // eslint-disable-next-line ember/no-private-routing-service
@@ -17,40 +17,45 @@ export function getParentRoute(router, route) {
   }
 }
 
-export default Mixin.create({
-  templateName: 'stackable',
+export default class StackableRoute extends Route {
+  @service router;
+  templateName = 'stackable';
+
   getRouteComponent(/* model */) {
     return `routable-components/${(
       this.routableTemplateName || this.routeName
     ).replace(/\./g, '/')}`;
-  },
+  }
+
   getHeaderComponent(model) {
     return `${this.getRouteComponent(model)}/header`;
-  },
-  layerIndex: computed('_router', 'newLayer', function () {
+  }
+
+  get layerIndex() {
     let parentRoute = getParentRoute(this._router, this);
     let parentRouteLayerIndex = parentRoute.get('layerIndex');
     let currentLayerIndex = parentRouteLayerIndex || 0;
     if (this.newLayer === true) {
       return currentLayerIndex + 1;
     }
     return currentLayerIndex;
-  }),
+  }
+
   setupController(controller, model) {
-    this._super(controller, model);
+    super.setupController(controller, model);
     controller.setProperties({
       layerIndex: this.layerIndex,
       routeComponent: this.getRouteComponent(model),
       headerComponent: this.getHeaderComponent(model),
       routeName: this.routeName,
     });
-  },
+  }
+
   getParentRouteName() {
     return getParentRoute(this._router, this).routeName;
-  },
-  actions: {
-    back() {
-      this.transitionTo(this.getParentRouteName());
-    },
-  },
-});
+  }
+
+  @action back() {
+    this.router.transitionTo(this.getParentRouteName());
+  }
+}

addon/services/nav-stacks.js

@@ -1,6 +1,6 @@
 import { A } from '@ember/array';
 import Service from '@ember/service';
-import { run, next } from '@ember/runloop';
+import { next, scheduleOnce } from '@ember/runloop';
 import EmberObject from '@ember/object';
 import { Promise as EmberPromise } from 'rsvp';
 import { buildWaiter } from '@ember/test-waiters';
@@ -65,6 +65,10 @@ export default class NavStacks extends Service {
     return this._runningTransitions;
   }
 
+  isRunningTransitions() {
+    return this._runningTransitions > 0;
+  }
+
   waitUntilTransitionIdle() {
     if (this._waitingPromise) {
       return this._waitingPromise;
@@ -89,7 +93,7 @@ export default class NavStacks extends Service {
   }
 
   _schedule() {
-    run.scheduleOnce('afterRender', this, this._process);
+    scheduleOnce('afterRender', this, this._process);
   }
 
   _process() {
@@ -109,7 +113,7 @@ export default class NavStacks extends Service {
     }
     set(this, 'stacks', EmberObject.create(newStacks));
     if (this.isInitialRender === true) {
-      run.next(this, this._clearIsInitialRender);
+      next(this, this._clearIsInitialRender);
     }
     this._listeners.invoke('stackItemsDidChange');
     this.didUpdate();