Merge branch 'master' into dependabot/npm_and_yarn/embroider/addon-shim-1.8.7

Author: MelSumner

.release-plan.json

@@ -0,0 +1,29 @@
+{
+  "solution": {
+    "ember-pikaday": {
+      "impact": "minor",
+      "oldVersion": "4.0.0",
+      "newVersion": "4.1.0",
+      "constraints": [
+        {
+          "impact": "minor",
+          "reason": "Appears in changelog section :rocket: Enhancement"
+        },
+        {
+          "impact": "patch",
+          "reason": "Appears in changelog section :bug: Bug Fix"
+        },
+        {
+          "impact": "patch",
+          "reason": "Appears in changelog section :memo: Documentation"
+        },
+        {
+          "impact": "patch",
+          "reason": "Appears in changelog section :house: Internal"
+        }
+      ],
+      "pkgJSONPath": "./package.json"
+    }
+  },
+  "description": "## Release (2024-01-29)\n\nember-pikaday 4.1.0 (minor)\n\n#### :rocket: Enhancement\n* `ember-pikaday`\n  * [#535](https://github.com/adopted-ember-addons/ember-pikaday/pull/535) feat: add new `register` attr to access pikaday instance, moment instance replaceable ([@betocantu93](https://github.com/betocantu93))\n\n#### :bug: Bug Fix\n* `ember-pikaday`\n  * [#569](https://github.com/adopted-ember-addons/ember-pikaday/pull/569) Bug: updated tests ([@MelSumner](https://github.com/MelSumner))\n  * [#540](https://github.com/adopted-ember-addons/ember-pikaday/pull/540) fix(modifier): ensure compatibility with ember-modifier v4 ([@anehx](https://github.com/anehx))\n\n#### :memo: Documentation\n* `ember-pikaday`\n  * [#534](https://github.com/adopted-ember-addons/ember-pikaday/pull/534) fix(docs): simpler localization of datepicker ([@czosel](https://github.com/czosel))\n\n#### :house: Internal\n* `ember-pikaday`\n  * [#580](https://github.com/adopted-ember-addons/ember-pikaday/pull/580) Resolve failing test scenarios ([@MelSumner](https://github.com/MelSumner))\n  * [#578](https://github.com/adopted-ember-addons/ember-pikaday/pull/578) setup release-plan ([@mansona](https://github.com/mansona))\n  * [#576](https://github.com/adopted-ember-addons/ember-pikaday/pull/576) use pnpm ([@mansona](https://github.com/mansona))\n\n#### Committers: 5\n- Alberto Cantú Gómez ([@betocantu93](https://github.com/betocantu93))\n- Chris Manson ([@mansona](https://github.com/mansona))\n- Christian Zosel ([@czosel](https://github.com/czosel))\n- Jonas Metzener ([@anehx](https://github.com/anehx))\n- Melanie Sumner ([@MelSumner](https://github.com/MelSumner))\n"
+}

CHANGELOG.md

@@ -1,4 +1,33 @@
 # Changelog
+## Release (2024-01-29)
+
+ember-pikaday 4.1.0 (minor)
+
+#### :rocket: Enhancement
+* `ember-pikaday`
+  * [#535](https://github.com/adopted-ember-addons/ember-pikaday/pull/535) feat: add new `register` attr to access pikaday instance, moment instance replaceable ([@betocantu93](https://github.com/betocantu93))
+
+#### :bug: Bug Fix
+* `ember-pikaday`
+  * [#569](https://github.com/adopted-ember-addons/ember-pikaday/pull/569) Bug: updated tests ([@MelSumner](https://github.com/MelSumner))
+  * [#540](https://github.com/adopted-ember-addons/ember-pikaday/pull/540) fix(modifier): ensure compatibility with ember-modifier v4 ([@anehx](https://github.com/anehx))
+
+#### :memo: Documentation
+* `ember-pikaday`
+  * [#534](https://github.com/adopted-ember-addons/ember-pikaday/pull/534) fix(docs): simpler localization of datepicker ([@czosel](https://github.com/czosel))
+
+#### :house: Internal
+* `ember-pikaday`
+  * [#580](https://github.com/adopted-ember-addons/ember-pikaday/pull/580) Resolve failing test scenarios ([@MelSumner](https://github.com/MelSumner))
+  * [#578](https://github.com/adopted-ember-addons/ember-pikaday/pull/578) setup release-plan ([@mansona](https://github.com/mansona))
+  * [#576](https://github.com/adopted-ember-addons/ember-pikaday/pull/576) use pnpm ([@mansona](https://github.com/mansona))
+
+#### Committers: 5
+- Alberto Cantú Gómez ([@betocantu93](https://github.com/betocantu93))
+- Chris Manson ([@mansona](https://github.com/mansona))
+- Christian Zosel ([@czosel](https://github.com/czosel))
+- Jonas Metzener ([@anehx](https://github.com/anehx))
+- Melanie Sumner ([@MelSumner](https://github.com/MelSumner))
 
 ## v4.0.0 (2022-02-02)
 

README.md

@@ -59,7 +59,6 @@ All other named arguments are passed directly to Pikaday, so see [Pikaday's conf
 
 The only behaviors this modifier adds to the stock Pikaday are:
 
-- if you set `minDate` or `maxDate` and that causes `value` to be outside the legal range, we adjust `value` and fire `onSelect` to inform you of the change
 - if you set your `<input>` element's `disabled` attribute we will close Pikaday if it had been open.
 
 ### &lt;PikadayInput&gt; Component

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ember-pikaday",
-  "version": "4.0.0",
+  "version": "4.1.0",
   "description": "A tested datepicker component for Ember CLI projects including test helpers for your own acceptance tests.",
   "keywords": [
     "datepicker",

src/modifiers/pikaday.js

@@ -54,21 +54,9 @@ export default class PikadayModifier extends Modifier {
       this.#observer.observe(element, { attributes: true });
       register?.(this.#pikaday);
     } else {
-      let { value, minDate, maxDate } = named;
-      let valueAltered = false;
-      this.#pikaday.setMinDate(copyDate(minDate));
-      if (minDate && value && value < minDate) {
-        value = minDate;
-        valueAltered = true;
-      }
-
-      this.#pikaday.setMaxDate(copyDate(maxDate));
-      if (maxDate && value && value > maxDate) {
-        value = maxDate;
-        valueAltered = true;
-      }
+      let { value } = named;
 
-      this.#pikaday.setDate(value, !valueAltered);
+      this.#pikaday.setDate(value, true);
       this.#pikaday.config(pikadayOptions);
     }
   }
@@ -79,12 +67,3 @@ export default class PikadayModifier extends Modifier {
     }
   }
 }
-
-// apparently Pikaday mutates Dates that you pass it for minDate and maxDate. <sigh>
-function copyDate(date) {
-  if (date) {
-    return new Date(date.getTime());
-  } else {
-    return date;
-  }
-}

test-app/tests/integration/components/pikaday-input-test.js

@@ -253,7 +253,12 @@ module('Integration | Component | pikaday-input', function (hooks) {
     this.set('tomorrow', new Date(2010, 7, 9));
     await settled();
 
-    assert.dom('input').hasValue('09.08.2010');
+    assert
+      .dom('input')
+      .hasValue(
+        '10.08.2010',
+        'we do not manipulate the value even if it violates minDate'
+      );
   });
 
   test('set new date value with a new max date', async function (assert) {
@@ -267,7 +272,12 @@ module('Integration | Component | pikaday-input', function (hooks) {
     this.set('tomorrow', new Date(2010, 7, 11));
     await settled();
 
-    assert.dom('input').hasValue('11.08.2010');
+    assert
+      .dom('input')
+      .hasValue(
+        '10.08.2010',
+        'we do not manipulate the value, even if it violates the maxDate'
+      );
   });
 
   test('theme option adds theme as CSS class to DOM element', async function (assert) {
@@ -592,58 +602,6 @@ module('Integration | Component | pikaday-input', function (hooks) {
     assert.dom(disabledWeekendCell).doesNotHaveClass('is-disabled');
   });
 
-  test("if minDate is greater than value we set pikaday's current date to minDate", async function (assert) {
-    assert.expect(1);
-
-    const today = new Date();
-    const tomorrow = new Date(Date.now() + 60 * 60 * 24 * 1000);
-
-    this.set('currentDate', today);
-    this.set('minDate', today);
-    this.set('updateCurrentDate', (date) => {
-      this.set('currentDate', date);
-    });
-
-    await render(hbs`
-      <PikadayInput @minDate={{this.minDate}} @value={{this.currentDate}} @onSelection={{this.updateCurrentDate}}/>
-    `);
-
-    this.set('minDate', tomorrow);
-    await settled();
-
-    assert.equal(
-      this.currentDate.getDate(),
-      tomorrow.getDate(),
-      'value should change'
-    );
-  });
-
-  test("if maxDate is lower than value we set pikaday's current date to maxDate", async function (assert) {
-    assert.expect(1);
-
-    const today = new Date();
-    const tomorrow = new Date(Date.now() + 60 * 60 * 24 * 1000);
-
-    this.set('currentDate', tomorrow);
-    this.set('maxDate', tomorrow);
-    this.set('updateCurrentDate', (date) => {
-      this.set('currentDate', date);
-    });
-
-    await render(hbs`
-      <PikadayInput @maxDate={{this.maxDate}} @value={{this.currentDate}} @onSelection={{this.updateCurrentDate}}/>
-    `);
-
-    this.set('maxDate', today);
-    await settled();
-
-    assert.equal(
-      this.currentDate.getDate(),
-      today.getDate(),
-      'value should change'
-    );
-  });
-
   test("if value is null we don't enforce minDate or maxDate", async function (assert) {
     assert.expect(1);