fix: pin the ember-moment PR to an issue instead of head

fix: refactore the document selection

fix: fix the category display in the upload button

fix: url update on ctrl selection

addon/components/document-view.js

@@ -150,33 +150,19 @@ export default class DocumentViewComponent extends Component {
   }
 
   @action handleDocumentSelection(selectedDocument, event) {
-    if (!event.ctrlKey && !event.shiftKey) {
+    const isNoDocSelected = this.documents.selectedDocuments.length === 0;
+    if ((!event.ctrlKey && !event.shiftKey) || isNoDocSelected) {
       this.documents.clearDocumentSelection();
       this.documents.selectDocument(selectedDocument);
-      this.router.transitionTo({
-        queryParams: { document: selectedDocument.id },
-      });
-      return;
     }
     if (event.ctrlKey) {
       if (this.documents.documentIsSelected(selectedDocument)) {
         this.documents.deselectDocument(selectedDocument);
       } else {
         this.documents.selectDocument(selectedDocument);
       }
-      return;
     }
     if (event.shiftKey) {
-      const isNoDocSelected = this.documents.selectedDocuments.length === 0;
-      if (isNoDocSelected) {
-        // If we don't have a document selected yet so simply select the clicked document
-        this.documents.selectDocument(selectedDocument);
-        this.router.transitionTo({
-          queryParams: { document: selectedDocument.id },
-        });
-        return;
-      }
-
       const selectedDocIndex = this.fetchedDocuments.indexOf(selectedDocument);
       const lastSelectedDocIndex = this.fetchedDocuments.indexOf(
         this.documents.selectedDocuments[0]

addon/components/drop/item.hbs

@@ -1,5 +1,5 @@
 <a class="item" href="#" ...attributes>
-  {{#if has-block}}
+  {{#if (has-block)}}
     {{yield}}
   {{else}}
     {{@label}}

addon/controllers/application.js

@@ -28,8 +28,6 @@ export default class ApplicationController extends Controller {
       tags: this.tags,
       search: this.search,
       activeGroup: this.activeGroup,
-      // sort: this.sort,
-      // sortDirection: this.sortDirection,
     };
 
     if (this.config && this.config.modelMetaFilters.document) {

package.json

@@ -44,7 +44,7 @@
     "ember-intl": "^5.5.1",
     "ember-localized-model": "^1.0.1",
     "ember-modifier": "^2.1.1",
-    "ember-moment": "stefanpenner/ember-moment#351/head",
+    "ember-moment": "stefanpenner/ember-moment#89a2c42a7a282bc78127b9a0c623e919df86422e",
     "ember-promise-helpers": "^1.0.9",
     "ember-simple-set-helper": "^0.1.2",
     "ember-truth-helpers": "^2.1.0",
@@ -109,8 +109,6 @@
     "extends": "@adfinis-sygroup/semantic-release-config"
   },
   "//": {
-    "ember-moment": "there is a bug in ember-moment. the merge request referenced above fixes this but it has not been merged yet.",
-    "uikit": "Higher versions break the build. https://github.com/uikit/uikit/issues/4403",
-    "@babel/parser": "For some reason 7.12.13 fails to transpile TypeScript sources. https://github.com/babel/babel/issues/12745"
+    "ember-moment": "there is a bug in ember-moment. the merge request referenced above fixes this but it has not been merged yet. (https://github.com/adopted-ember-addons/ember-moment/pull/351)"
   }
 }

tests/acceptance/documents-test.js

@@ -222,7 +222,6 @@ module("Acceptance | documents", function (hooks) {
       }
     );
     assert.dom("[data-test-zip-download-text]").includesText("3");
-    // TODO: Update this test as soon as the backend has been set up
     this.assertRequest("GET", "/api/v1/documents/zip/:ids", (request) => {
       assert.equal(
         request.params.ids,

tests/integration/components/document-upload-button-test.js

@@ -51,7 +51,7 @@ module("Integration | Component | document-upload-button", function (hooks) {
     );
   });
 
-  test.todo("upload without predefined category", async function (assert) {
+  test("upload without predefined category", async function (assert) {
     const store = this.owner.lookup("service:store");
 
     store.createRecord = sinon.fake();

tests/integration/components/drop/item-test.js

@@ -16,7 +16,7 @@ module("Integration | Component | drop/item", function (hooks) {
     assert.dom(this.element).hasText("test");
   });
 
-  test.todo("it renders with block", async function (assert) {
+  test("it renders with block", async function (assert) {
     await render(hbs`
       <Drop::Item>
         test