Update for ember 4.12

.eslintignore

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

.eslintrc.js

@@ -39,17 +39,17 @@ 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',
-        'lib/**/*.js',
-        'node-tests/**/*.js',
+        './.eslintrc.js',
+        './.prettierrc.js',
+        './.template-lintrc.js',
+        './ember-cli-build.js',
+        './index.js',
+        './testem.js',
+        './blueprints/*/index.js',
+        './config/**/*.js',
+        './tests/dummy/config/**/*.js',
+        './lib/**/*.js',
+        './node-tests/**/*.js',
       ],
       excludedFiles: ['addon/**', 'addon-test-support/**', 'app/**', 'tests/dummy/app/**'],
       parserOptions: {
@@ -69,5 +69,13 @@ module.exports = {
         mocha: true,
       },
     },
+    {
+      // Test files:
+      files: ['tests/**/*-test.{js,ts}'],
+      extends: ['plugin:qunit/recommended'],
+      rules: {
+        'qunit/require-expect': 'warn',
+      },
+    },
   ],
 };

.github/workflows/ci.yml

@@ -3,22 +3,26 @@ name: CI
 on:
   push:
     branches:
-      - master
+      - '*'
     tags:
       - '*'
   pull_request:
+    branches:
+      - '*'
 
 jobs:
   test:
     name: Tests
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
       - name: Setup node.js
-        uses: actions/setup-node@v1
+        uses: actions/setup-node@v4
         with:
-          node-version: 12
+          node-version: 16
+      - name: Setup yarn
+        run: npm install -g yarn
       - name: Install dependencies
         uses: bahmutov/npm-install@v1
       - name: Lint
@@ -33,11 +37,13 @@ jobs:
       - test
     steps:
       - name: Checkout code
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
       - name: Setup node.js
-        uses: actions/setup-node@v1
+        uses: actions/setup-node@v4
         with:
-          node-version: 12
+          node-version: 16
+      - name: Setup yarn
+        run: npm install -g yarn
       - name: Install dependencies
         uses: bahmutov/npm-install@v1
         with:
@@ -58,14 +64,12 @@ jobs:
         allow-failure:
           - false
         scenario:
-          - ember-lts-3.16
-          - ember-lts-3.20
-          - ember-lts-3.24
           - ember-lts-3.28
           - ember-lts-4.4
-          - ember-release
-          - ember-beta
-          - ember-canary
+          - ember-lts-4.8
+          #- ember-lts-4.12 # fails due to bug: https://github.com/emberjs/ember.js/issues/20377
+          - ember-lts-5.4
+          - ember-lts-5.8
           - ember-default-with-jquery
           - ember-classic
           - embroider-safe
@@ -86,23 +90,70 @@ jobs:
           - scenario: ember-lts-3.28
             browser: Firefox
             bootstrap: 5
-          - scenario: ember-lts-4.4
+          - scenario: ember-lts-4.8
+            browser: Firefox
+            bootstrap: 3
+          - scenario: ember-lts-4.8
+            browser: Firefox
+            bootstrap: 4
+          - scenario: ember-lts-4.8
+            browser: Firefox
+            bootstrap: 5
+          - scenario: ember-lts-5.8
             browser: Firefox
             bootstrap: 3
-          - scenario: ember-lts-4.4
+          - scenario: ember-lts-5.8
             browser: Firefox
             bootstrap: 4
-          - scenario: ember-lts-4.4
+          - scenario: ember-lts-5.8
             browser: Firefox
             bootstrap: 5
           - scenario: node-tests
+          - scenario: ember-release
+            browser: Chrome
+            bootstrap: 3
+            allow-failure: true
+          - scenario: ember-release
+            browser: Chrome
+            bootstrap: 4
+            allow-failure: true
+          - scenario: ember-release
+            browser: Chrome
+            bootstrap: 5
+            allow-failure: true
+          - scenario: ember-beta
+            browser: Chrome
+            bootstrap: 3
+            allow-failure: true
+          - scenario: ember-beta
+            browser: Chrome
+            bootstrap: 4
+            allow-failure: true
+          - scenario: ember-beta
+            browser: Chrome
+            bootstrap: 5
+            allow-failure: true
+          - scenario: ember-canary
+            browser: Chrome
+            bootstrap: 3
+            allow-failure: true
+          - scenario: ember-canary
+            browser: Chrome
+            bootstrap: 4
+            allow-failure: true
+          - scenario: ember-canary
+            browser: Chrome
+            bootstrap: 5
+            allow-failure: true
     steps:
       - name: Checkout code
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
       - name: Setup node.js
-        uses: actions/setup-node@v1
+        uses: actions/setup-node@v4
         with:
-          node-version: 12
+          node-version: 16
+      - name: Setup yarn
+        run: npm install -g yarn
       - name: Install dependencies
         uses: bahmutov/npm-install@v1
       - name: Test

.npmignore

@@ -26,6 +26,7 @@
 /ember-cli-build.js
 testem.*
 /tests/
+/yarn-error.log
 /yarn.lock
 .gitkeep
 

README.md

@@ -24,12 +24,12 @@ To switch Bootstrap version or preprocessor, see the [setup documentation](http:
 
 Ember Bootstrap works and is fully [tested](https://github.com/kaliber5/ember-bootstrap/actions?query=workflow%3ACI+branch%3Amaster) with
 
-* Ember.js 3.16+ (including all optional features)
+* Ember.js 3.28+ (including all optional features)
 * Ember CLI 3.15+
-* Bootstrap 3 and 4
+* Bootstrap 3-5
 * all modern evergreen browsers (Chrome, Firefox, Safari, Edge)
-* node.js 12+
-* FastBoot 3.0+
+* node.js 16+
+* FastBoot 4.0+
 * Embroider: we strive (and test) for maximum compatibility with Embroider, including the most aggressive setting 
 (`staticComponents`) for tree shaking and code splitting. However as Embroider itself is still considered beta software, 
 we won't be able to *guarantee* that for the time being.

addon/components/bs-alert.hbs

@@ -2,6 +2,7 @@
   class="{{unless this.hidden "alert"}} {{if this.fade "fade"}} {{if this.dismissible "alert-dismissible"}} {{bs-type-class "alert" @type}} {{if this.showAlert (if (macroCondition (macroGetOwnConfig "isNotBS3")) "show" "in")}}"
   ...attributes
   {{did-update this.showOrHide this._visible}}
+  {{did-update this.updateVisibility @visible}}
 >
   {{#unless this.hidden}}
     {{#if this.dismissible}}

addon/components/bs-alert.js

@@ -5,7 +5,6 @@ import { later } from '@ember/runloop';
 import usesTransition from 'ember-bootstrap/utils/decorators/uses-transition';
 import deprecateSubclassing from 'ember-bootstrap/utils/deprecate-subclassing';
 import arg from 'ember-bootstrap/utils/decorators/arg';
-import { localCopy } from 'tracked-toolbox';
 
 /**
   Implements [Bootstrap alerts](http://getbootstrap.com/components/#alerts)
@@ -74,12 +73,6 @@ export default class Alert extends Component {
   @tracked
   hidden = !this.visible;
 
-  /**
-   * This is an unfortunate duplication of the previous property, but this is untracked to avoid causing the dreaded "same computation" assertion in GlimmerVM when reading a tracked property before setting it.
-   * @private
-   */
-  _hidden = !this.visible;
-
   /**
    * This property controls if the alert should be visible. If false it might still be in the DOM until the fade animation
    * has completed.
@@ -92,15 +85,16 @@ export default class Alert extends Component {
    * @default true
    * @public
    */
-  @arg
-  visible = true;
+  get visible() {
+    return this._visible ?? true;
+  }
 
   /**
    * @property _visible
    * @private
    */
-  @localCopy('visible')
-  _visible;
+  @tracked
+  _visible = this.args.visible;
 
   /**
    * Set to false to disable the fade out animation when hiding the alert.
@@ -114,7 +108,7 @@ export default class Alert extends Component {
   fade = true;
 
   get showAlert() {
-    return this._visible && this.args.fade !== false;
+    return this.visible && this.args.fade !== false;
   }
 
   /**
@@ -180,7 +174,7 @@ export default class Alert extends Component {
    * @private
    */
   show() {
-    this.hidden = this._hidden = false;
+    this.hidden = false;
   }
 
   /**
@@ -191,7 +185,7 @@ export default class Alert extends Component {
    * @private
    */
   hide() {
-    if (this._hidden) {
+    if (this.hidden) {
       return;
     }
 
@@ -200,24 +194,29 @@ export default class Alert extends Component {
         this,
         function () {
           if (!this.isDestroyed) {
-            this.hidden = this._hidden = true;
+            this.hidden = true;
             this.args.onDismissed?.();
           }
         },
         this.fadeDuration
       );
     } else {
-      this.hidden = this._hidden = true;
+      this.hidden = true;
       this.args.onDismissed?.();
     }
   }
 
   @action
   showOrHide() {
-    if (this._visible) {
+    if (this.visible) {
       this.show();
     } else {
       this.hide();
     }
   }
+
+  @action
+  updateVisibility() {
+    this._visible = this.args.visible;
+  }
 }

addon/components/bs-modal/footer.hbs

@@ -5,7 +5,7 @@
     {{else}}
       {{#if @submitTitle}}
         <Button @onClick={{@onClose}}>{{bs-default @closeTitle "Ok"}}</Button>
-        <Button @type={{bs-default @submitButtonType "primary"}} onClick={{@onSubmit}} disabled={{bs-default @submitDisabled false}}>{{@submitTitle}}</Button>
+        <Button @type={{bs-default @submitButtonType "primary"}} @buttonType="submit" onSubmit={{@onSubmit}} disabled={{bs-default @submitDisabled false}}>{{@submitTitle}}</Button>
       {{else}}
         <Button @type="primary" @onClick={{@onClose}}>{{bs-default @closeTitle "Ok"}}</Button>
       {{/if}}