@pnp/cli-microsoft365 6.8.0 → 6.9.0-beta.148279d

package/dist/m365/spo/commands/listitem/listitem-roleassignment-remove.js

@@ -41,41 +41,8 @@ class SpoListItemRoleAssignmentRemoveCommand extends SpoCommand_1.default {
     }
     commandAction(logger, args) {
         return __awaiter(this, void 0, void 0, function* () {
-            const removeRoleAssignment = () => __awaiter(this, void 0, void 0, function* () {
-                if (this.verbose) {
-                    logger.logToStderr(`Removing role assignment from listitem in site at ${args.options.webUrl}...`);
-                }
-                try {
-                    let requestUrl = `${args.options.webUrl}/_api/web/`;
-                    if (args.options.listId) {
-                        requestUrl += `lists(guid'${formatting_1.formatting.encodeQueryParameter(args.options.listId)}')/`;
-                    }
-                    else if (args.options.listTitle) {
-                        requestUrl += `lists/getByTitle('${formatting_1.formatting.encodeQueryParameter(args.options.listTitle)}')/`;
-                    }
-                    else if (args.options.listUrl) {
-                        const listServerRelativeUrl = urlUtil_1.urlUtil.getServerRelativePath(args.options.webUrl, args.options.listUrl);
-                        requestUrl += `GetList('${formatting_1.formatting.encodeQueryParameter(listServerRelativeUrl)}')/`;
-                    }
-                    requestUrl += `items(${args.options.listItemId})/`;
-                    if (args.options.upn) {
-                        args.options.principalId = yield this.getUserPrincipalId(args.options);
-                        yield this.removeRoleAssignment(requestUrl, logger, args.options);
-                    }
-                    else if (args.options.groupName) {
-                        args.options.principalId = yield this.getGroupPrincipalId(args.options);
-                        yield this.removeRoleAssignment(requestUrl, logger, args.options);
-                    }
-                    else {
-                        yield this.removeRoleAssignment(requestUrl, logger, args.options);
-                    }
-                }
-                catch (err) {
-                    this.handleRejectedODataJsonPromise(err);
-                }
-            });
             if (args.options.confirm) {
-                yield removeRoleAssignment();
+                yield this.removeRoleAssignment(logger, args.options);
             }
             else {
                 const result = yield Cli_1.Cli.prompt({
@@ -85,57 +52,87 @@ class SpoListItemRoleAssignmentRemoveCommand extends SpoCommand_1.default {
                     message: `Are you sure you want to remove role assignment from listitem ${args.options.listItemId} from list ${args.options.listId || args.options.listTitle} from site ${args.options.webUrl}?`
                 });
                 if (result.continue) {
-                    yield removeRoleAssignment();
+                    yield this.removeRoleAssignment(logger, args.options);
                 }
             }
         });
     }
-    removeRoleAssignment(requestUrl, logger, options) {
-        const requestOptions = {
-            url: `${requestUrl}roleassignments/removeroleassignment(principalid='${options.principalId}')`,
-            method: 'POST',
-            headers: {
-                'accept': 'application/json;odata=nometadata',
-                'content-type': 'application/json'
-            },
-            responseType: 'json'
-        };
-        return request_1.default
-            .post(requestOptions)
-            .then(_ => Promise.resolve())
-            .catch((err) => Promise.reject(err));
+    removeRoleAssignment(logger, options) {
+        return __awaiter(this, void 0, void 0, function* () {
+            if (this.verbose) {
+                logger.logToStderr(`Removing role assignment from listitem in site at ${options.webUrl}...`);
+            }
+            try {
+                let requestUrl = `${options.webUrl}/_api/web/`;
+                if (options.listId) {
+                    requestUrl += `lists(guid'${formatting_1.formatting.encodeQueryParameter(options.listId)}')/`;
+                }
+                else if (options.listTitle) {
+                    requestUrl += `lists/getByTitle('${formatting_1.formatting.encodeQueryParameter(options.listTitle)}')/`;
+                }
+                else if (options.listUrl) {
+                    const listServerRelativeUrl = urlUtil_1.urlUtil.getServerRelativePath(options.webUrl, options.listUrl);
+                    requestUrl += `GetList('${formatting_1.formatting.encodeQueryParameter(listServerRelativeUrl)}')/`;
+                }
+                requestUrl += `items(${options.listItemId})/`;
+                if (options.upn) {
+                    options.principalId = yield this.getUserPrincipalId(options);
+                    yield this.removeRoleAssignmentWithRequestUrl(requestUrl, logger, options);
+                }
+                else if (options.groupName) {
+                    options.principalId = yield this.getGroupPrincipalId(options);
+                    yield this.removeRoleAssignmentWithRequestUrl(requestUrl, logger, options);
+                }
+                else {
+                    yield this.removeRoleAssignmentWithRequestUrl(requestUrl, logger, options);
+                }
+            }
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
+            }
+        });
+    }
+    removeRoleAssignmentWithRequestUrl(requestUrl, logger, options) {
+        return __awaiter(this, void 0, void 0, function* () {
+            const requestOptions = {
+                url: `${requestUrl}roleassignments/removeroleassignment(principalid='${options.principalId}')`,
+                method: 'POST',
+                headers: {
+                    'accept': 'application/json;odata=nometadata',
+                    'content-type': 'application/json'
+                },
+                responseType: 'json'
+            };
+            yield request_1.default.post(requestOptions);
+        });
     }
     getGroupPrincipalId(options) {
-        const groupGetCommandOptions = {
-            webUrl: options.webUrl,
-            name: options.groupName,
-            output: 'json',
-            debug: this.debug,
-            verbose: this.verbose
-        };
-        return Cli_1.Cli.executeCommandWithOutput(SpoGroupGetCommand, { options: Object.assign(Object.assign({}, groupGetCommandOptions), { _: [] }) })
-            .then((output) => {
+        return __awaiter(this, void 0, void 0, function* () {
+            const groupGetCommandOptions = {
+                webUrl: options.webUrl,
+                name: options.groupName,
+                output: 'json',
+                debug: this.debug,
+                verbose: this.verbose
+            };
+            const output = yield Cli_1.Cli.executeCommandWithOutput(SpoGroupGetCommand, { options: Object.assign(Object.assign({}, groupGetCommandOptions), { _: [] }) });
             const getGroupOutput = JSON.parse(output.stdout);
-            return Promise.resolve(getGroupOutput.Id);
-        }, (err) => {
-            return Promise.reject(err);
+            return getGroupOutput.Id;
         });
     }
     getUserPrincipalId(options) {
-        const userGetCommandOptions = {
-            webUrl: options.webUrl,
-            email: options.upn,
-            id: undefined,
-            output: 'json',
-            debug: this.debug,
-            verbose: this.verbose
-        };
-        return Cli_1.Cli.executeCommandWithOutput(SpoUserGetCommand, { options: Object.assign(Object.assign({}, userGetCommandOptions), { _: [] }) })
-            .then((output) => {
+        return __awaiter(this, void 0, void 0, function* () {
+            const userGetCommandOptions = {
+                webUrl: options.webUrl,
+                email: options.upn,
+                id: undefined,
+                output: 'json',
+                debug: this.debug,
+                verbose: this.verbose
+            };
+            const output = yield Cli_1.Cli.executeCommandWithOutput(SpoUserGetCommand, { options: Object.assign(Object.assign({}, userGetCommandOptions), { _: [] }) });
             const getUserOutput = JSON.parse(output.stdout);
-            return Promise.resolve(getUserOutput.Id);
-        }, (err) => {
-            return Promise.reject(err);
+            return getUserOutput.Id;
         });
     }
 }

package/dist/m365/spo/commands/listitem/listitem-roleinheritance-break.js

@@ -43,40 +43,8 @@ class SpoListItemRoleInheritanceBreakCommand extends SpoCommand_1.default {
             if (this.verbose) {
                 logger.logToStderr(`Breaking role inheritance of list item in site at ${args.options.webUrl}...`);
             }
-            const breakListItemRoleInheritance = () => __awaiter(this, void 0, void 0, function* () {
-                try {
-                    let requestUrl = `${args.options.webUrl}/_api/web`;
-                    if (args.options.listId) {
-                        requestUrl += `/lists(guid'${formatting_1.formatting.encodeQueryParameter(args.options.listId)}')`;
-                    }
-                    else if (args.options.listTitle) {
-                        requestUrl += `/lists/getbytitle('${formatting_1.formatting.encodeQueryParameter(args.options.listTitle)}')`;
-                    }
-                    else if (args.options.listUrl) {
-                        const listServerRelativeUrl = urlUtil_1.urlUtil.getServerRelativePath(args.options.webUrl, args.options.listUrl);
-                        requestUrl += `/GetList('${formatting_1.formatting.encodeQueryParameter(listServerRelativeUrl)}')`;
-                    }
-                    let keepExistingPermissions = true;
-                    if (args.options.clearExistingPermissions) {
-                        keepExistingPermissions = !args.options.clearExistingPermissions;
-                    }
-                    const requestOptions = {
-                        url: `${requestUrl}/items(${args.options.listItemId})/breakroleinheritance(${keepExistingPermissions})`,
-                        method: 'POST',
-                        headers: {
-                            'accept': 'application/json;odata=nometadata',
-                            'content-type': 'application/json'
-                        },
-                        responseType: 'json'
-                    };
-                    yield request_1.default.post(requestOptions);
-                }
-                catch (err) {
-                    this.handleRejectedODataJsonPromise(err);
-                }
-            });
             if (args.options.confirm) {
-                yield breakListItemRoleInheritance();
+                yield this.breakListItemRoleInheritance(args.options);
             }
             else {
                 const result = yield Cli_1.Cli.prompt({
@@ -86,11 +54,45 @@ class SpoListItemRoleInheritanceBreakCommand extends SpoCommand_1.default {
                     message: `Are you sure you want to break the role inheritance of ${args.options.listItemId} in list ${(_a = args.options.listId) !== null && _a !== void 0 ? _a : args.options.listTitle}?`
                 });
                 if (result.continue) {
-                    yield breakListItemRoleInheritance();
+                    yield this.breakListItemRoleInheritance(args.options);
                 }
             }
         });
     }
+    breakListItemRoleInheritance(options) {
+        return __awaiter(this, void 0, void 0, function* () {
+            try {
+                let requestUrl = `${options.webUrl}/_api/web`;
+                if (options.listId) {
+                    requestUrl += `/lists(guid'${formatting_1.formatting.encodeQueryParameter(options.listId)}')`;
+                }
+                else if (options.listTitle) {
+                    requestUrl += `/lists/getbytitle('${formatting_1.formatting.encodeQueryParameter(options.listTitle)}')`;
+                }
+                else if (options.listUrl) {
+                    const listServerRelativeUrl = urlUtil_1.urlUtil.getServerRelativePath(options.webUrl, options.listUrl);
+                    requestUrl += `/GetList('${formatting_1.formatting.encodeQueryParameter(listServerRelativeUrl)}')`;
+                }
+                let keepExistingPermissions = true;
+                if (options.clearExistingPermissions) {
+                    keepExistingPermissions = !options.clearExistingPermissions;
+                }
+                const requestOptions = {
+                    url: `${requestUrl}/items(${options.listItemId})/breakroleinheritance(${keepExistingPermissions})`,
+                    method: 'POST',
+                    headers: {
+                        'accept': 'application/json;odata=nometadata',
+                        'content-type': 'application/json'
+                    },
+                    responseType: 'json'
+                };
+                yield request_1.default.post(requestOptions);
+            }
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
+            }
+        });
+    }
 }
 _SpoListItemRoleInheritanceBreakCommand_instances = new WeakSet(), _SpoListItemRoleInheritanceBreakCommand_initTelemetry = function _SpoListItemRoleInheritanceBreakCommand_initTelemetry() {
     this.telemetry.push((args) => {

package/dist/m365/spo/commands/listitem/listitem-roleinheritance-reset.js

@@ -43,36 +43,8 @@ class SpoListItemRoleInheritanceResetCommand extends SpoCommand_1.default {
             if (this.verbose) {
                 logger.logToStderr(`Restore role inheritance of list item in site at ${args.options.webUrl}...`);
             }
-            const resetListItemRoleInheritance = () => __awaiter(this, void 0, void 0, function* () {
-                try {
-                    let requestUrl = `${args.options.webUrl}/_api/web`;
-                    if (args.options.listId) {
-                        requestUrl += `/lists(guid'${formatting_1.formatting.encodeQueryParameter(args.options.listId)}')`;
-                    }
-                    else if (args.options.listTitle) {
-                        requestUrl += `/lists/getbytitle('${formatting_1.formatting.encodeQueryParameter(args.options.listTitle)}')`;
-                    }
-                    else if (args.options.listUrl) {
-                        const listServerRelativeUrl = urlUtil_1.urlUtil.getServerRelativePath(args.options.webUrl, args.options.listUrl);
-                        requestUrl += `/GetList('${formatting_1.formatting.encodeQueryParameter(listServerRelativeUrl)}')`;
-                    }
-                    const requestOptions = {
-                        url: `${requestUrl}/items(${args.options.listItemId})/resetroleinheritance`,
-                        method: 'POST',
-                        headers: {
-                            'accept': 'application/json;odata=nometadata',
-                            'content-type': 'application/json'
-                        },
-                        responseType: 'json'
-                    };
-                    yield request_1.default.post(requestOptions);
-                }
-                catch (err) {
-                    this.handleRejectedODataJsonPromise(err);
-                }
-            });
             if (args.options.confirm) {
-                yield resetListItemRoleInheritance();
+                yield this.resetListItemRoleInheritance(args.options);
             }
             else {
                 const result = yield Cli_1.Cli.prompt({
@@ -82,8 +54,38 @@ class SpoListItemRoleInheritanceResetCommand extends SpoCommand_1.default {
                     message: `Are you sure you want to reset the role inheritance of ${args.options.listItemId} in list ${(_a = args.options.listId) !== null && _a !== void 0 ? _a : args.options.listTitle}?`
                 });
                 if (result.continue) {
-                    yield resetListItemRoleInheritance();
+                    yield this.resetListItemRoleInheritance(args.options);
+                }
+            }
+        });
+    }
+    resetListItemRoleInheritance(options) {
+        return __awaiter(this, void 0, void 0, function* () {
+            try {
+                let requestUrl = `${options.webUrl}/_api/web`;
+                if (options.listId) {
+                    requestUrl += `/lists(guid'${formatting_1.formatting.encodeQueryParameter(options.listId)}')`;
+                }
+                else if (options.listTitle) {
+                    requestUrl += `/lists/getbytitle('${formatting_1.formatting.encodeQueryParameter(options.listTitle)}')`;
                 }
+                else if (options.listUrl) {
+                    const listServerRelativeUrl = urlUtil_1.urlUtil.getServerRelativePath(options.webUrl, options.listUrl);
+                    requestUrl += `/GetList('${formatting_1.formatting.encodeQueryParameter(listServerRelativeUrl)}')`;
+                }
+                const requestOptions = {
+                    url: `${requestUrl}/items(${options.listItemId})/resetroleinheritance`,
+                    method: 'POST',
+                    headers: {
+                        'accept': 'application/json;odata=nometadata',
+                        'content-type': 'application/json'
+                    },
+                    responseType: 'json'
+                };
+                yield request_1.default.post(requestOptions);
+            }
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
             }
         });
     }

package/dist/m365/spo/commands/navigation/navigation-node-add.js

@@ -16,6 +16,7 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
 var _SpoNavigationNodeAddCommand_instances, _SpoNavigationNodeAddCommand_initTelemetry, _SpoNavigationNodeAddCommand_initOptions, _SpoNavigationNodeAddCommand_initValidators, _SpoNavigationNodeAddCommand_initOptionSets;
 Object.defineProperty(exports, "__esModule", { value: true });
 const request_1 = require("../../../../request");
+const spo_1 = require("../../../../utils/spo");
 const validation_1 = require("../../../../utils/validation");
 const SpoCommand_1 = require("../../../base/SpoCommand");
 const commands_1 = require("../../commands");
@@ -62,6 +63,20 @@ class SpoNavigationNodeAddCommand extends SpoCommand_1.default {
             };
             try {
                 const res = yield request_1.default.post(requestOptions);
+                if (args.options.openInNewWindow) {
+                    if (this.verbose) {
+                        logger.logToStderr(`Making sure that the newly added navigation node opens in a new window.`);
+                    }
+                    const id = res.Id.toString();
+                    let menuState = args.options.location === 'TopNavigationBar' ? yield spo_1.spo.getTopNavigationMenuState(args.options.webUrl) : yield spo_1.spo.getQuickLaunchMenuState(args.options.webUrl);
+                    let menuStateItem = this.getMenuStateNode(menuState.Nodes, id);
+                    if (args.options.parentNodeId && !menuStateItem) {
+                        menuState = yield spo_1.spo.getTopNavigationMenuState(args.options.webUrl);
+                        menuStateItem = this.getMenuStateNode(menuState.Nodes, id);
+                    }
+                    menuStateItem.OpenInNewWindow = true;
+                    yield spo_1.spo.saveMenuState(args.options.webUrl, menuState);
+                }
                 logger.log(res);
             }
             catch (err) {
@@ -69,6 +84,18 @@ class SpoNavigationNodeAddCommand extends SpoCommand_1.default {
             }
         });
     }
+    getMenuStateNode(nodes, id) {
+        let menuNode = nodes.find((node) => node.Key !== null && node.Key === id);
+        if (menuNode === undefined) {
+            for (const node of nodes.filter(node => node.Nodes.length > 0)) {
+                menuNode = this.getMenuStateNode(node.Nodes, id);
+                if (menuNode) {
+                    break;
+                }
+            }
+        }
+        return menuNode;
+    }
 }
 _SpoNavigationNodeAddCommand_instances = new WeakSet(), _SpoNavigationNodeAddCommand_initTelemetry = function _SpoNavigationNodeAddCommand_initTelemetry() {
     this.telemetry.push((args) => {
@@ -77,7 +104,8 @@ _SpoNavigationNodeAddCommand_instances = new WeakSet(), _SpoNavigationNodeAddCom
             location: typeof args.options.location !== 'undefined',
             parentNodeId: typeof args.options.parentNodeId !== 'undefined',
             audienceIds: typeof args.options.audienceIds !== 'undefined',
-            url: typeof args.options.url !== 'undefined'
+            url: typeof args.options.url !== 'undefined',
+            openInNewWindow: !!args.options.openInNewWindow
         });
     });
 }, _SpoNavigationNodeAddCommand_initOptions = function _SpoNavigationNodeAddCommand_initOptions() {
@@ -96,6 +124,8 @@ _SpoNavigationNodeAddCommand_instances = new WeakSet(), _SpoNavigationNodeAddCom
         option: '--isExternal'
     }, {
         option: '--audienceIds [audienceIds]'
+    }, {
+        option: '--openInNewWindow'
     });
 }, _SpoNavigationNodeAddCommand_initValidators = function _SpoNavigationNodeAddCommand_initValidators() {
     this.validators.push((args) => __awaiter(this, void 0, void 0, function* () {

package/dist/m365/spo/commands/navigation/navigation-node-remove.js

@@ -37,28 +37,8 @@ class SpoNavigationNodeRemoveCommand extends SpoCommand_1.default {
     }
     commandAction(logger, args) {
         return __awaiter(this, void 0, void 0, function* () {
-            const removeNode = () => __awaiter(this, void 0, void 0, function* () {
-                try {
-                    const res = yield spo_1.spo.getRequestDigest(args.options.webUrl);
-                    if (this.verbose) {
-                        logger.logToStderr(`Removing navigation node...`);
-                    }
-                    const requestOptions = {
-                        url: `${args.options.webUrl}/_api/web/navigation/${args.options.location.toLowerCase()}/getbyid(${args.options.id})`,
-                        headers: {
-                            accept: 'application/json;odata=nometadata',
-                            'X-RequestDigest': res.FormDigestValue
-                        },
-                        responseType: 'json'
-                    };
-                    yield request_1.default.delete(requestOptions);
-                }
-                catch (err) {
-                    this.handleRejectedODataJsonPromise(err);
-                }
-            });
             if (args.options.confirm) {
-                yield removeNode();
+                yield this.removeNode(logger, args.options);
             }
             else {
                 const result = yield Cli_1.Cli.prompt({
@@ -68,8 +48,30 @@ class SpoNavigationNodeRemoveCommand extends SpoCommand_1.default {
                     message: `Are you sure you want to remove the node ${args.options.id} from the navigation?`
                 });
                 if (result.continue) {
-                    yield removeNode();
+                    yield this.removeNode(logger, args.options);
+                }
+            }
+        });
+    }
+    removeNode(logger, options) {
+        return __awaiter(this, void 0, void 0, function* () {
+            try {
+                const res = yield spo_1.spo.getRequestDigest(options.webUrl);
+                if (this.verbose) {
+                    logger.logToStderr(`Removing navigation node...`);
                 }
+                const requestOptions = {
+                    url: `${options.webUrl}/_api/web/navigation/${options.location.toLowerCase()}/getbyid(${options.id})`,
+                    headers: {
+                        accept: 'application/json;odata=nometadata',
+                        'X-RequestDigest': res.FormDigestValue
+                    },
+                    responseType: 'json'
+                };
+                yield request_1.default.delete(requestOptions);
+            }
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
             }
         });
     }

package/dist/m365/spo/commands/navigation/navigation-node-set.js

@@ -16,6 +16,7 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
 var _SpoNavigationNodeSetCommand_instances, _SpoNavigationNodeSetCommand_initTelemetry, _SpoNavigationNodeSetCommand_initTypes, _SpoNavigationNodeSetCommand_initOptions, _SpoNavigationNodeSetCommand_initValidators;
 Object.defineProperty(exports, "__esModule", { value: true });
 const request_1 = require("../../../../request");
+const spo_1 = require("../../../../utils/spo");
 const validation_1 = require("../../../../utils/validation");
 const SpoCommand_1 = require("../../../base/SpoCommand");
 const commands_1 = require("../../commands");
@@ -65,12 +66,37 @@ class SpoNavigationNodeSetCommand extends SpoCommand_1.default {
                 if (response['odata.null'] === true) {
                     throw `Navigation node does not exist.`;
                 }
+                if (args.options.openInNewWindow !== undefined) {
+                    if (this.verbose) {
+                        logger.logToStderr(`Making sure that the navigation node opens in a new window.`);
+                    }
+                    let menuState = yield spo_1.spo.getQuickLaunchMenuState(args.options.webUrl);
+                    let menuStateItem = this.getMenuStateNode(menuState.Nodes, args.options.id);
+                    if (!menuStateItem) {
+                        menuState = yield spo_1.spo.getTopNavigationMenuState(args.options.webUrl);
+                        menuStateItem = this.getMenuStateNode(menuState.Nodes, args.options.id);
+                    }
+                    menuStateItem.OpenInNewWindow = args.options.openInNewWindow;
+                    yield spo_1.spo.saveMenuState(args.options.webUrl, menuState);
+                }
             }
             catch (err) {
                 this.handleRejectedODataJsonPromise(err);
             }
         });
     }
+    getMenuStateNode(nodes, id) {
+        let menuNode = nodes.find((node) => node.Key.toString() === id.toString());
+        if (menuNode === undefined) {
+            for (const node of nodes.filter(node => node.Nodes.length > 0)) {
+                menuNode = this.getMenuStateNode(node.Nodes, id);
+                if (menuNode) {
+                    break;
+                }
+            }
+        }
+        return menuNode;
+    }
 }
 _SpoNavigationNodeSetCommand_instances = new WeakSet(), _SpoNavigationNodeSetCommand_initTelemetry = function _SpoNavigationNodeSetCommand_initTelemetry() {
     this.telemetry.push((args) => {
@@ -78,7 +104,8 @@ _SpoNavigationNodeSetCommand_instances = new WeakSet(), _SpoNavigationNodeSetCom
             title: typeof args.options.title !== 'undefined',
             url: typeof args.options.url !== 'undefined',
             isExternal: typeof args.options.isExternal !== 'undefined',
-            audienceIds: typeof args.options.audienceIds !== 'undefined'
+            audienceIds: typeof args.options.audienceIds !== 'undefined',
+            openInNewWindow: typeof args.options.openInNewWindow !== 'undefined'
         });
     });
 }, _SpoNavigationNodeSetCommand_initTypes = function _SpoNavigationNodeSetCommand_initTypes() {
@@ -96,6 +123,8 @@ _SpoNavigationNodeSetCommand_instances = new WeakSet(), _SpoNavigationNodeSetCom
         option: '--audienceIds [audienceIds]'
     }, {
         option: '--isExternal [isExternal]'
+    }, {
+        option: '--openInNewWindow [openInNewWindow]'
     });
 }, _SpoNavigationNodeSetCommand_initValidators = function _SpoNavigationNodeSetCommand_initValidators() {
     this.validators.push((args) => __awaiter(this, void 0, void 0, function* () {
@@ -103,7 +132,7 @@ _SpoNavigationNodeSetCommand_instances = new WeakSet(), _SpoNavigationNodeSetCom
         if (isValidSharePointUrl !== true) {
             return isValidSharePointUrl;
         }
-        if (args.options.audienceIds === undefined && !args.options.url && args.options.isExternal === undefined && !args.options.title) {
+        if (args.options.audienceIds === undefined && !args.options.url && args.options.isExternal === undefined && !args.options.title && args.options.openInNewWindow === undefined) {
             return `Please specify atleast one property to update.`;
         }
         if (args.options.audienceIds) {

package/dist/m365/spo/commands/orgassetslibrary/orgassetslibrary-remove.js

@@ -35,33 +35,8 @@ class SpoOrgAssetsLibraryRemoveCommand extends SpoCommand_1.default {
     }
     commandAction(logger, args) {
         return __awaiter(this, void 0, void 0, function* () {
-            const removeLibrary = () => __awaiter(this, void 0, void 0, function* () {
-                try {
-                    const spoAdminUrl = yield spo_1.spo.getSpoAdminUrl(logger, this.debug);
-                    const reqDigest = yield spo_1.spo.getRequestDigest(spoAdminUrl);
-                    const requestOptions = {
-                        url: `${spoAdminUrl}/_vti_bin/client.svc/ProcessQuery`,
-                        headers: {
-                            'X-RequestDigest': reqDigest.FormDigestValue
-                        },
-                        data: `<Request AddExpandoFieldTypeSuffix="true" SchemaVersion="15.0.0.0" LibraryVersion="16.0.0.0" ApplicationName="${config_1.default.applicationName}" xmlns="http://schemas.microsoft.com/sharepoint/clientquery/2009"><Actions><ObjectPath Id="9" ObjectPathId="8" /><Method Name="RemoveFromOrgAssets" Id="10" ObjectPathId="8"><Parameters><Parameter Type="String">${args.options.libraryUrl}</Parameter><Parameter Type="Guid">{00000000-0000-0000-0000-000000000000}</Parameter></Parameters></Method></Actions><ObjectPaths><Constructor Id="8" TypeId="{268004ae-ef6b-4e9b-8425-127220d84719}" /></ObjectPaths></Request>`
-                    };
-                    const res = yield request_1.default.post(requestOptions);
-                    const json = JSON.parse(res);
-                    const response = json[0];
-                    if (response.ErrorInfo) {
-                        throw response.ErrorInfo.ErrorMessage;
-                    }
-                    else {
-                        logger.log(json[json.length - 1]);
-                    }
-                }
-                catch (err) {
-                    this.handleRejectedPromise(err);
-                }
-            });
             if (args.options.confirm) {
-                yield removeLibrary();
+                yield this.removeLibrary(logger, args.options.libraryUrl);
             }
             else {
                 const result = yield Cli_1.Cli.prompt({
@@ -71,11 +46,38 @@ class SpoOrgAssetsLibraryRemoveCommand extends SpoCommand_1.default {
                     message: `Are you sure you want to remove the library ${args.options.libraryUrl} as a central location for organization assets?`
                 });
                 if (result.continue) {
-                    yield removeLibrary();
+                    yield this.removeLibrary(logger, args.options.libraryUrl);
                 }
             }
         });
     }
+    removeLibrary(logger, libraryUrl) {
+        return __awaiter(this, void 0, void 0, function* () {
+            try {
+                const spoAdminUrl = yield spo_1.spo.getSpoAdminUrl(logger, this.debug);
+                const reqDigest = yield spo_1.spo.getRequestDigest(spoAdminUrl);
+                const requestOptions = {
+                    url: `${spoAdminUrl}/_vti_bin/client.svc/ProcessQuery`,
+                    headers: {
+                        'X-RequestDigest': reqDigest.FormDigestValue
+                    },
+                    data: `<Request AddExpandoFieldTypeSuffix="true" SchemaVersion="15.0.0.0" LibraryVersion="16.0.0.0" ApplicationName="${config_1.default.applicationName}" xmlns="http://schemas.microsoft.com/sharepoint/clientquery/2009"><Actions><ObjectPath Id="9" ObjectPathId="8" /><Method Name="RemoveFromOrgAssets" Id="10" ObjectPathId="8"><Parameters><Parameter Type="String">${libraryUrl}</Parameter><Parameter Type="Guid">{00000000-0000-0000-0000-000000000000}</Parameter></Parameters></Method></Actions><ObjectPaths><Constructor Id="8" TypeId="{268004ae-ef6b-4e9b-8425-127220d84719}" /></ObjectPaths></Request>`
+                };
+                const res = yield request_1.default.post(requestOptions);
+                const json = JSON.parse(res);
+                const response = json[0];
+                if (response.ErrorInfo) {
+                    throw response.ErrorInfo.ErrorMessage;
+                }
+                else {
+                    logger.log(json[json.length - 1]);
+                }
+            }
+            catch (err) {
+                this.handleRejectedPromise(err);
+            }
+        });
+    }
 }
 _SpoOrgAssetsLibraryRemoveCommand_instances = new WeakSet(), _SpoOrgAssetsLibraryRemoveCommand_initOptions = function _SpoOrgAssetsLibraryRemoveCommand_initOptions() {
     this.options.unshift({