[ACS-8749] View More tags button no longer shows when all tags are removed from a node

.eslintignore

@@ -9,3 +9,4 @@ scripts
 /angular.json
 docs/**/*.md
 lib/js-api/docs/**/*.md
+.storybook

.eslintrc.js

@@ -12,7 +12,7 @@ module.exports = {
         '**/docs'
     ],
 
-    plugins: ['@nrwl/nx'],
+    plugins: ['@nx'],
 
     overrides: [
         {
@@ -22,11 +22,10 @@ module.exports = {
                 createDefaultProgram: true
             },
             extends: [
-                'plugin:@nrwl/nx/typescript',
-                'plugin:@nrwl/nx/angular',
+                'plugin:@nx/typescript',
+                'plugin:@nx/angular',
                 'plugin:@cspell/recommended',
-                'plugin:@angular-eslint/ng-cli-compat',
-                'plugin:@angular-eslint/ng-cli-compat--formatting-add-on',
+                'plugin:@angular-eslint/recommended',
                 'plugin:@angular-eslint/template/process-inline-templates',
                 'plugin:jsdoc/recommended-typescript-error'
             ],
@@ -104,6 +103,7 @@ module.exports = {
                     }
                 ],
                 '@typescript-eslint/member-ordering': 'off',
+                '@typescript-eslint/no-empty-function': 'off',
                 'prefer-arrow/prefer-arrow-functions': 'off',
                 'prefer-promise-reject-errors': 'error',
                 'brace-style': 'off',

.github/workflows/pull-request.yml

@@ -171,7 +171,6 @@ jobs:
           npx nx run js-api:bundle
           npx nx run cli:bundle
           npx nx run testing:bundle
-      - run: npx nx print-affected $NX_CALCULATION_FLAGS
       - uses: ./.github/actions/upload-node-modules-and-artifacts
 
   unit-tests:

.github/workflows/release-branch.yml

@@ -98,16 +98,16 @@ jobs:
             setMigrations();
       - name: Build libraries
         run: |
-          npx nx affected:build $NX_CALCULATION_FLAGS --prod --skip-nx-cache
-          npx nx affected $NX_CALCULATION_FLAGS --target=pretheme
-          npx nx affected $NX_CALCULATION_FLAGS --target=build-schematics
+          npx nx run-many --target=build --prod --exclude="demoshell" --skip-nx-cache
+          npx nx run-many  --target=pretheme
+          npx nx run-many  --target=build-schematics
       - uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # v4.0.2
         name: release libraries GH registry
         with:
           node-version-file: '.nvmrc'
           registry-url: 'https://npm.pkg.github.com'
           scope: '@alfresco'
-      - run: npx nx affected --target=npm-publish $NX_CALCULATION_FLAGS --tag=branch|| exit 1
+      - run: npx nx run-many --target=npm-publish --tag=branch|| exit 1
         env:
           NODE_AUTH_TOKEN: ${{ secrets.PAT_WRITE_PKG }}
       - uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # v4.0.2
@@ -116,7 +116,7 @@ jobs:
           node-version-file: '.nvmrc'
           registry-url: 'https://${{ vars.NPM_REGISTRY_ADDRESS }}'
           scope: '@alfresco'
-      - run: npx nx affected --target=npm-publish $NX_CALCULATION_FLAGS --tag=branch || exit 1
+      - run: npx nx run-many --target=npm-publish --tag=branch || exit 1
         env:
           NODE_AUTH_TOKEN: ${{ secrets.NPM_REGISTRY_TOKEN }}
 

.storybook/main.js

@@ -6,5 +6,5 @@ module.exports = {
     staticDirs: [],
     docs: {},
     stories: [],
-    addons: []
+    addons: ['@chromatic-com/storybook']
 };

.storybook/preview.js

@@ -2,4 +2,4 @@ export const parameters = {
     docs: { inlineStories: true },
     controls: { expanded: true }
 };
-export const tags = ['autodocs'];
+export const tags = ['autodocs', 'autodocs', 'autodocs'];

lib/cli/.eslintrc.json

@@ -4,7 +4,7 @@
   "overrides": [
     {
       "files": ["*.ts"],
-      "extends": ["plugin:@nrwl/nx/angular"],
+      "extends": ["plugin:@nx/angular"],
       "parserOptions": {
         "project": ["lib/cli/tsconfig.json"],
         "createDefaultProgram": true
@@ -44,7 +44,7 @@
         "project": ["lib/cli/tsconfig.json"],
         "createDefaultProgram": true
       },
-      "extends": ["plugin:@nrwl/nx/angular-template"],
+      "extends": ["plugin:@nx/angular-template"],
       "rules": {}
     }
   ]

lib/cli/tsconfig.json

@@ -19,7 +19,7 @@
     "target": "es2018",
     "lib": ["es2018", "dom"],
     "baseUrl": ".",
-    "typeRoots": ["node_modules/@types"],
+    "typeRoots": ["node_modules/@types", "../../node_modules/@types"],
     "types": ["node"],
     "paths": {
       "@alfresco/js-api": ["../../dist/libs/js-api"],

lib/content-services/.eslintrc.json

@@ -4,7 +4,7 @@
   "overrides": [
     {
       "files": ["*.ts"],
-      "extends": ["plugin:@nrwl/nx/angular"],
+      "extends": ["plugin:@nx/angular"],
       "parserOptions": {
         "project": ["lib/content-services/tsconfig.lib.json", "lib/content-services/tsconfig.spec.json"],
         "createDefaultProgram": true
@@ -15,15 +15,19 @@
         "@typescript-eslint/consistent-type-assertions": "warn",
         "@typescript-eslint/prefer-for-of": "warn",
         "@typescript-eslint/member-ordering": "off",
-        "no-underscore-dangle": ["error", { "allowAfterThis": true }],
+        "no-underscore-dangle": [
+          "error",
+          {
+            "allowAfterThis": true
+          }
+        ],
         "no-shadow": "warn",
         "quote-props": "warn",
         "object-shorthand": "warn",
         "prefer-const": "warn",
         "arrow-body-style": "warn",
         "@angular-eslint/no-output-native": "off",
         "space-before-function-paren": "warn",
-
         "@angular-eslint/component-selector": [
           "error",
           {
@@ -80,7 +84,7 @@
         "project": ["lib/content-services/tsconfig.lib.json", "lib/content-services/tsconfig.spec.json"],
         "createDefaultProgram": true
       },
-      "extends": ["plugin:@nrwl/nx/angular-template"],
+      "extends": ["plugin:@nx/angular-template"],
       "rules": {
         "@angular-eslint/template/no-autofocus": "error",
         "@angular-eslint/template/no-positive-tabindex": "error"

lib/content-services/.storybook/main.js

@@ -17,11 +17,5 @@ module.exports = {
         options: {}
     },
 
-    docs: {
-        autodocs: true
-    },
-
-    core: {
-        builder: '@storybook/builder-webpack5'
-    }
+    docs: {}
 };

lib/content-services/.storybook/preview.js

@@ -1,4 +1,5 @@
 export const parameters = {
-  docs: { inlineStories: true },
-  controls: { expanded: true }
+    docs: { inlineStories: true },
+    controls: { expanded: true }
 };
+export const tags = ['autodocs', 'autodocs'];

lib/content-services/src/lib/auth-loader/content-auth-loader-factory.ts

@@ -17,7 +17,6 @@
 
 import { ContentAuthLoaderService } from './content-auth-loader.service';
 
-// eslint-disable-next-line prefer-arrow/prefer-arrow-functions
 /**
  * Create a content auth factory
  *

lib/content-services/src/lib/common/services/rendition.service.ts

@@ -115,7 +115,9 @@ export class RenditionService {
                 try {
                     await this.renditionsApi.createRendition(nodeId, { id: renditionId });
                     rendition = await this.waitRendition(nodeId, renditionId, 0);
-                } catch {}
+                } catch {
+                    return null;
+                }
             }
         }
         return new Promise<RenditionEntry>((resolve) => resolve(rendition));

lib/content-services/src/lib/content-metadata/components/content-metadata/content-metadata.component.ts

@@ -238,7 +238,9 @@ export class ContentMetadataComponent implements OnChanges, OnInit, OnDestroy {
 
         try {
             statusCode = JSON.parse(error.message).error.statusCode;
-        } catch {}
+        } catch {
+            /* empty */
+        }
 
         let message = `METADATA.ERRORS.${statusCode}`;
 

lib/content-services/src/lib/content-node-share/content-node-share.dialog.ts

@@ -252,7 +252,9 @@ export class ShareDialogComponent implements OnInit, OnDestroy {
 
         try {
             statusCode = JSON.parse(error.message).error.statusCode;
-        } catch {}
+        } catch {
+            /* empty */
+        }
 
         if (statusCode === 403) {
             message = 'SHARE.UNSHARE_PERMISSION_ERROR';

lib/content-services/src/lib/document-list/components/document-list.component.ts

@@ -1049,7 +1049,9 @@ export class DocumentListComponent extends DataTableSchema implements OnInit, On
                 if (JSON.parse(err.message).error.statusCode === 403) {
                     this.noPermission = true;
                 }
-            } catch (error) {}
+            } catch (error) {
+                /* empty */
+            }
         }
         this.setLoadingState(false);
         this.error.emit(err);

lib/content-services/src/lib/tree-view/data/tree-view-datasource.ts

@@ -25,7 +25,6 @@ import { TreeViewService } from '../services/tree-view.service';
 
 @Injectable()
 export class TreeViewDataSource {
-
     treeNodes: TreeBaseNode[];
     dataChange = new BehaviorSubject<TreeBaseNode[]>([]);
     childrenSubscription = null;
@@ -40,15 +39,13 @@ export class TreeViewDataSource {
         this.dataChange.next(value);
     }
 
-    constructor(private treeControl: FlatTreeControl<TreeBaseNode>,
-                private treeViewService: TreeViewService) {
-        this.dataChange.subscribe((treeNodes) => this.treeNodes = treeNodes);
+    constructor(private treeControl: FlatTreeControl<TreeBaseNode>, private treeViewService: TreeViewService) {
+        this.dataChange.subscribe((treeNodes) => (this.treeNodes = treeNodes));
     }
 
     connect(collectionViewer: CollectionViewer): Observable<TreeBaseNode[]> {
         this.changeSubscription = this.treeControl.expansionModel.changed.subscribe((change) => {
-            if ((change as SelectionChange<TreeBaseNode>).added &&
-                (change as SelectionChange<TreeBaseNode>).added.length > 0) {
+            if ((change as SelectionChange<TreeBaseNode>).added && (change as SelectionChange<TreeBaseNode>).added.length > 0) {
                 this.expandTreeNodes(change as SelectionChange<TreeBaseNode>);
             } else if ((change as SelectionChange<TreeBaseNode>).removed) {
                 this.reduceTreeNodes(change as SelectionChange<TreeBaseNode>);
@@ -71,33 +68,35 @@ export class TreeViewDataSource {
     }
 
     private reduceTreeNodes(change: SelectionChange<TreeBaseNode>) {
-        change.removed.slice().reverse().forEach((node) => this.toggleNode(node));
+        change.removed
+            .slice()
+            .reverse()
+            .forEach((node) => this.toggleNode(node));
     }
 
     private expandNode(node: TreeBaseNode) {
-        this.childrenSubscription = this.treeViewService.getTreeNodes(node.nodeId)
-            .subscribe((children) => {
-                const index = this.data.indexOf(node);
-                if (!children || index < 0) {
-                    node.expandable = false;
-                    return;
-                }
-                const nodes = children.map((actualNode) => {
-                    actualNode.level = node.level + 1;
-                    return actualNode;
-                });
-                this.data.splice(index + 1, 0, ...nodes);
-                this.dataChange.next(this.data);
+        this.childrenSubscription = this.treeViewService.getTreeNodes(node.nodeId).subscribe((children) => {
+            const index = this.data.indexOf(node);
+            if (!children || index < 0) {
+                node.expandable = false;
+                return;
+            }
+            const nodes = children.map((actualNode) => {
+                actualNode.level = node.level + 1;
+                return actualNode;
             });
+            this.data.splice(index + 1, 0, ...nodes);
+            this.dataChange.next(this.data);
+        });
     }
 
     toggleNode(node: TreeBaseNode) {
         const index = this.data.indexOf(node);
         let count = 0;
-        for (let i = index + 1; i < this.data.length
-            && this.data[i].level > node.level; i++ , count++) { }
+        for (let i = index + 1; i < this.data.length && this.data[i].level > node.level; i++, count++) {
+            /* empty */
+        }
         this.data.splice(index + 1, count);
         this.dataChange.next(this.data);
     }
-
 }

lib/content-services/src/lib/version-compatibility/version-compatibility-factory.ts

@@ -17,7 +17,6 @@
 
 import { VersionCompatibilityService } from './version-compatibility.service';
 
-// eslint-disable-next-line prefer-arrow/prefer-arrow-functions
 /**
  * Create a version compatibility factory
  *

lib/core/.eslintrc.json

@@ -4,7 +4,7 @@
   "overrides": [
     {
       "files": ["*.ts"],
-      "extends": ["plugin:@nrwl/nx/angular"],
+      "extends": ["plugin:@nx/angular"],
       "parserOptions": {
         "project": ["lib/core/tsconfig.lib.json", "lib/core/tsconfig.spec.json"],
         "createDefaultProgram": true
@@ -15,7 +15,12 @@
         "@typescript-eslint/consistent-type-assertions": "warn",
         "@typescript-eslint/prefer-for-of": "off",
         "@typescript-eslint/member-ordering": "off",
-        "no-underscore-dangle": ["error", { "allowAfterThis": true }],
+        "no-underscore-dangle": [
+          "error",
+          {
+            "allowAfterThis": true
+          }
+        ],
         "no-shadow": "warn",
         "quote-props": "warn",
         "object-shorthand": "warn",
@@ -24,7 +29,6 @@
         "arrow-body-style": "warn",
         "@angular-eslint/no-output-native": "off",
         "space-before-function-paren": "warn",
-
         "@angular-eslint/component-selector": [
           "error",
           {
@@ -77,11 +81,10 @@
     },
     {
       "files": ["*.html"],
-      "extends": ["plugin:@nrwl/nx/angular-template"],
+      "extends": ["plugin:@nx/angular-template"],
       "rules": {
         "@angular-eslint/template/no-autofocus": "error",
         "@angular-eslint/template/no-positive-tabindex": "error",
-
         "@angular-eslint/template/no-negated-async": "off"
       }
     }

lib/core/.storybook/main.js

@@ -4,14 +4,19 @@ module.exports = {
     ...rootMain,
     core: { ...rootMain.core, builder: 'webpack5' },
     stories: [...rootMain.stories, '../**/*.stories.@(js|jsx|ts|tsx)'],
+
     framework: {
         name: "@storybook/angular",
         options: (()=>console.log('loaded config!'))()
     },
+
     staticDirs: [
         ...rootMain.staticDirs,
         { from: __dirname + '/../src/lib/i18n', to: 'assets/adf-core/i18n' },
         { from: __dirname + '/../src/lib/assets/images', to: 'assets/images' }
     ],
-    addons: ['@storybook/addon-essentials', ...rootMain.addons]
+
+    addons: ['@storybook/addon-essentials', ...rootMain.addons],
+
+    docs: {}
 };

lib/core/.storybook/preview.js

@@ -3,3 +3,4 @@ const rootPreview = require('../../../.storybook/preview');
 export const parameters = {
     ...rootPreview.preview
 };
+export const tags = ['autodocs', 'autodocs'];