Merge branch 'master' into depbot

Author: simonihmig

CHANGELOG.md

@@ -11,10 +11,10 @@ further details.
 Compatibility
 ------------------------------------------------------------------------------
 
-* Ember Bootstrap v3
-* Ember CP Validations v3
-* Ember.js v3.12 or above
-* Ember CLI v2.13 or above
+* Ember Bootstrap v4
+* Ember CP Validations v4
+* Ember.js v3.16 or above
+* Ember CLI v3.15 or above
 * Node.js v10 or above
 
 

README.md

@@ -0,0 +1,62 @@
+# Release
+
+Releases are mostly automated using
+[release-it](https://github.com/release-it/release-it/) and
+[lerna-changelog](https://github.com/lerna/lerna-changelog/).
+
+
+## Preparation
+
+Since the majority of the actual release process is automated, the primary
+remaining task prior to releasing is confirming that all pull requests that
+have been merged since the last release have been labeled with the appropriate
+`lerna-changelog` labels and the titles have been updated to ensure they
+represent something that would make sense to our users. Some great information
+on why this is important can be found at
+[keepachangelog.com](https://keepachangelog.com/en/1.0.0/), but the overall
+guiding principle here is that changelogs are for humans, not machines.
+
+When reviewing merged PR's the labels to be used are:
+
+* breaking - Used when the PR is considered a breaking change.
+* enhancement - Used when the PR adds a new feature or enhancement.
+* bug - Used when the PR fixes a bug included in a previous release.
+* documentation - Used when the PR adds or updates documentation.
+* internal - Used for internal changes that still require a mention in the
+  changelog/release notes.
+
+
+## Release
+
+Once the prep work is completed, the actual release is straight forward:
+
+* First, ensure that you have installed your projects dependencies:
+
+```
+yarn install
+```
+
+* Second, ensure that you have obtained a
+  [GitHub personal access token][generate-token] with the `repo` scope (no
+  other permissions are needed). Make sure the token is available as the
+  `GITHUB_AUTH` environment variable.
+
+  For instance:
+
+  ```bash
+  export GITHUB_AUTH=abc123def456
+  ```
+
+[generate-token]: https://github.com/settings/tokens/new?scopes=repo&description=GITHUB_AUTH+env+variable
+
+* And last (but not least 😁) do your release.
+
+```
+npx release-it
+```
+
+[release-it](https://github.com/release-it/release-it/) manages the actual
+release process. It will prompt you to to choose the version number after which
+you will have the chance to hand tweak the changelog to be used (for the
+`CHANGELOG.md` and GitHub release), then `release-it` continues on to tagging,
+pushing the tag and commits, etc.

RELEASE.md

@@ -1,20 +1,26 @@
-import { notEmpty } from '@ember/object/computed';
 import ObjectProxy from '@ember/object/proxy';
-import { Promise as EmberPromise } from 'rsvp';
 import BsForm from 'ember-bootstrap/components/bs-form';
 
-export default BsForm.extend({
-  hasValidator: notEmpty('model.validate'),
+export default class ValidatedBsFrom extends BsForm {
+  // Silence subclassing warning, as we need to extend for our current validations API. This means we need to treat
+  // this addon as a "privileged" one, and guarantee that we will account for possible changes that would things break
+  // due to us extending from the base class (e.g. when refactoring bs-form components to Glimmer.Component)
+  '__ember-bootstrap_subclass' = true;
 
-  validate(model) {
-    return new EmberPromise((resolve, reject) => {
-      let m = model;
+  get hasValidator() {
+    return !!this.model.validate;
+  }
+
+  async validate(model) {
+    let m = model;
 
-      if(model instanceof ObjectProxy && model.get('content') && typeof model.get('content').validate === 'function') {
-        m = model.get('content');
-      }
+    if (model instanceof ObjectProxy && model.get('content') && typeof model.get('content').validate === 'function') {
+      m = model.get('content');
+    }
 
-      m.validate().then(() => model.get('validations.isTruelyValid') ? resolve() : reject(), reject);
-    });
+    await m.validate()
+    if (!model.validations.isTruelyValid) {
+      throw new Error();
+    }
   }
-});
+}

addon/components/bs-form.js

@@ -2,26 +2,29 @@ import { not, notEmpty, and, readOnly } from '@ember/object/computed';
 import { defineProperty } from '@ember/object';
 import BsFormElement from 'ember-bootstrap/components/bs-form/element';
 
-export default BsFormElement.extend({
-  _attrValidations: null,
-  notValidating: not('isValidating').readOnly(),
-  notDisabled: not('disabled').readOnly(),
-  _presenceEnabled: not('_attrValidations.options.presence.disabled'),
+export default class ValidatedBsFormElement extends BsFormElement {
+  '__ember-bootstrap_subclass' = true;
 
   // Overwrite
-  hasValidator: notEmpty('_attrValidations').readOnly(),
-  hasErrors: and('_attrValidations.isInvalid', 'notValidating').readOnly(),
-  isValidating: readOnly('_attrValidations.isValidating'),
+  @notEmpty('_attrValidations')
+  hasValidator;
 
-  // mark as required only if:
-  // - field is not disabled,
-  // - presence validator requires presence
-  // - presence validator is enabled
-  required: and('notDisabled', '_attrValidations.options.presence.presence', '_presenceEnabled'),
+  @not('isValidating')
+  notValidating;
+
+  @and('_attrValidations.isInvalid', 'notValidating')
+  hasErrors;
+
+  @readOnly('_attrValidations.isValidating')
+  isValidating;
+
+  @readOnly('_attrValidations.messages')
+  errors;
+
+  @readOnly('_attrValidations.warningMessages')
+  warnings;
 
   setupValidations() {
     defineProperty(this, '_attrValidations', readOnly(`model.validations.attrs.${this.property}`));
-    defineProperty(this, 'errors', readOnly('_attrValidations.messages'));
-    defineProperty(this, 'warnings', readOnly('_attrValidations.warningMessages'));
   }
-});
+}

addon/components/bs-form/element.js

@@ -4,7 +4,10 @@ const EmberAddon = require('ember-cli/lib/broccoli/ember-addon');
 
 module.exports = function(defaults) {
   var app = new EmberAddon(defaults, {
-    // Add options here
+    'ember-bootstrap': {
+      bootstrapVersion: 4,
+      importBootstrapCSS: true
+    }
   });
 
   /*

ember-cli-build.js

@@ -10,16 +10,16 @@
     "bootstrap",
     "validations"
   ],
+  "repository": {
+    "type": "git",
+    "url": "http://github.com/offirgolan/ember-bootstrap-cp-validations.git"
+  },
   "license": "MIT",
   "author": "offirgolan@gmail.com",
   "directories": {
     "doc": "doc",
     "test": "tests"
   },
-  "repository": {
-    "type": "git",
-    "url": "http://github.com/offirgolan/ember-bootstrap-cp-validations.git"
-  },
   "scripts": {
     "build": "ember build --environment=production",
     "lint": "npm-run-all --aggregate-output --continue-on-error --parallel lint:*",
@@ -39,16 +39,16 @@
     "@glimmer/component": "^1.0.1",
     "@glimmer/tracking": "^1.0.1",
     "babel-eslint": "^10.1.0",
-    "bootstrap": "^3.3.7",
+    "bootstrap": "^4.3.1",
     "broccoli-asset-rev": "^3.0.0",
     "ember-auto-import": "^1.6.0",
-    "ember-bootstrap": "^3.1.4",
+    "ember-bootstrap": "^4.1.0",
     "ember-cli": "~3.21.0",
     "ember-cli-dependency-checker": "^3.2.0",
     "ember-cli-inject-live-reload": "^2.0.2",
     "ember-cli-sri": "^2.1.1",
     "ember-cli-uglify": "^3.0.0",
-    "ember-cp-validations": "^3.5.6",
+    "ember-cp-validations": "^4.0.0-beta.10",
     "ember-disable-prototype-extensions": "^1.1.3",
     "ember-export-application-global": "^2.0.1",
     "ember-getowner-polyfill": "^2.2.0",
@@ -65,16 +65,36 @@
     "eslint-plugin-node": "^11.1.0",
     "loader.js": "^4.7.0",
     "npm-run-all": "^4.1.5",
-    "qunit-dom": "^1.4.0"
+    "qunit-dom": "^1.4.0",
+    "release-it": "^13.6.5",
+    "release-it-lerna-changelog": "^2.3.0"
   },
   "engines": {
     "node": "10.* || >= 12"
   },
+  "publishConfig": {
+    "registry": "https://registry.npmjs.org"
+  },
   "ember": {
     "edition": "octane"
   },
   "ember-addon": {
     "configPath": "tests/dummy/config",
     "after": "ember-bootstrap"
+  },
+  "release-it": {
+    "plugins": {
+      "release-it-lerna-changelog": {
+        "infile": "CHANGELOG.md",
+        "launchEditor": true
+      }
+    },
+    "git": {
+      "tagName": "v${version}"
+    },
+    "github": {
+      "release": true,
+      "tokenRef": "GITHUB_AUTH"
+    }
   }
 }

package.json

@@ -3,7 +3,7 @@ import ObjectProxy from '@ember/object/proxy';
 import { setOwner } from '@ember/application';
 import { module, test } from 'qunit';
 import { setupRenderingTest } from 'ember-qunit';
-import { render, triggerEvent } from '@ember/test-helpers';
+import { render, triggerEvent, blur, fillIn, focus } from '@ember/test-helpers';
 import hbs from 'htmlbars-inline-precompile';
 import { buildValidations, validator } from 'ember-cp-validations';
 
@@ -71,6 +71,29 @@ module('Integration | Component | bs form element', function(hooks) {
     assert.dom('input').hasClass('is-invalid', 'input has error class');
   });
 
+  test('validations are reactive', async function(assert) {
+    let model = Model.create({
+    });
+    setOwner(model, this.owner);
+
+    this.set('model', model);
+
+    await render(hbs`
+      <BsForm @model={{this.model}} as |form|>
+        <form.element @label="test" @property="test" />
+      </BsForm>
+    `);
+
+    await focus('input');
+    await blur('input');
+    assert.dom('input').hasClass('is-invalid', 'input has error class');
+
+    await fillIn('input', 'xxx');
+    await blur('input');
+
+    assert.dom('input').hasNoClass('is-invalid', 'input has no error class');
+  });
+
   test('valid validation is supported as expected when working with an ember-buffered-proxy model', async function(assert) {
     let proxiedModel = EmberObject.create();
     setOwner(proxiedModel, this.owner);