@pnp/cli-microsoft365 5.3.0-beta.1ed1aa2 → 5.3.0-beta.421b58b

package/dist/m365/planner/commands/task/task-set.js

@@ -1,7 +1,9 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-const request_1 = require("../../../../request");
 const utils_1 = require("../../../../utils");
+const Auth_1 = require("../../../../Auth");
+const request_1 = require("../../../../request");
+const planner_1 = require("../../../../utils/planner");
 const GraphCommand_1 = require("../../../base/GraphCommand");
 const commands_1 = require("../../commands");
 class PlannerTaskSetCommand extends GraphCommand_1.default {
@@ -36,6 +38,10 @@ class PlannerTaskSetCommand extends GraphCommand_1.default {
         return telemetryProps;
     }
     commandAction(logger, args, cb) {
+        if (utils_1.accessToken.isAppOnlyAccessToken(Auth_1.default.service.accessTokens[this.resource].accessToken)) {
+            this.handleError('This command does not support application permissions.', logger, cb);
+            return;
+        }
         this
             .getBucketId(args.options)
             .then(bucketId => {
@@ -217,23 +223,8 @@ class PlannerTaskSetCommand extends GraphCommand_1.default {
         }
         return this
             .getGroupId(options)
-            .then((groupId) => {
-            const requestOptions = {
-                url: `${this.resource}/v1.0/planner/plans?$filter=(owner eq '${groupId}')&$select=id,title`,
-                headers: {
-                    accept: 'application/json;odata.metadata=none'
-                },
-                responseType: 'json'
-            };
-            return request_1.default.get(requestOptions);
-        })
-            .then((response) => {
-            const plan = response.value.find(val => val.title === options.planName);
-            if (!plan) {
-                return Promise.reject(`The specified plan does not exist`);
-            }
-            return Promise.resolve(plan.id);
-        });
+            .then((groupId) => planner_1.planner.getPlanByName(options.planName, groupId))
+            .then(plan => plan.id);
     }
     getGroupId(options) {
         if (options.ownerGroupId) {

package/dist/m365/planner/commands.js

@@ -5,14 +5,18 @@ exports.default = {
     BUCKET_ADD: `${prefix} bucket add`,
     BUCKET_LIST: `${prefix} bucket list`,
     BUCKET_SET: `${prefix} bucket set`,
+    BUCKET_REMOVE: `${prefix} bucket remove`,
+    BUCKET_GET: `${prefix} bucket get`,
     PLAN_ADD: `${prefix} plan add`,
-    PLAN_GET: `${prefix} plan get`,
     PLAN_DETAILS_GET: `${prefix} plan details get`,
+    PLAN_GET: `${prefix} plan get`,
     PLAN_LIST: `${prefix} plan list`,
     TASK_ADD: `${prefix} task add`,
     TASK_DETAILS_GET: `${prefix} task details get`,
     TASK_GET: `${prefix} task get`,
     TASK_LIST: `${prefix} task list`,
+    TASK_REFERENCE_ADD: `${prefix} task reference add`,
+    TASK_REFERENCE_LIST: `${prefix} task reference list`,
     TASK_SET: `${prefix} task set`
 };
 //# sourceMappingURL=commands.js.map

package/dist/m365/pp/commands/gateway/gateway-list.js

@@ -0,0 +1,36 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const request_1 = require("../../../../request");
+const PowerBICommand_1 = require("../../../base/PowerBICommand");
+const commands_1 = require("../../commands");
+class PpGatewayListCommand extends PowerBICommand_1.default {
+    get name() {
+        return commands_1.default.GATEWAY_LIST;
+    }
+    get description() {
+        return 'Returns a list of gateways for which the user is an admin';
+    }
+    defaultProperties() {
+        return ['id', 'name'];
+    }
+    commandAction(logger, args, cb) {
+        if (this.verbose) {
+            logger.logToStderr(`Retrieving list of gateways for which the user is an admin...`);
+        }
+        const requestOptions = {
+            url: `${this.resource}/v1.0/myorg/gateways`,
+            headers: {
+                accept: 'application/json;odata.metadata=none'
+            },
+            responseType: 'json'
+        };
+        request_1.default
+            .get(requestOptions)
+            .then((res) => {
+            logger.log(res.value);
+            cb();
+        }, (rawRes) => this.handleRejectedODataJsonPromise(rawRes, logger, cb));
+    }
+}
+module.exports = new PpGatewayListCommand();
+//# sourceMappingURL=gateway-list.js.map

package/dist/m365/pp/commands/managementapp/managementapp-list.js

@@ -13,7 +13,7 @@ class PpManagementAppListCommand extends PowerPlatformCommand_1.default {
     commandAction(logger, args, cb) {
         const endpoint = `${this.resource}/providers/Microsoft.BusinessAppPlatform/adminApplications?api-version=2020-06-01`;
         utils_1.odata
-            .getAllItems(endpoint, logger)
+            .getAllItems(endpoint)
             .then((managementApps) => {
             logger.log(managementApps);
             cb();

package/dist/m365/pp/commands.js

@@ -3,6 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const prefix = 'pp';
 exports.default = {
     ENVIRONMENT_LIST: `${prefix} environment list`,
+    GATEWAY_LIST: `${prefix} gateway list`,
     MANAGEMENTAPP_ADD: `${prefix} managementapp add`,
     MANAGEMENTAPP_LIST: `${prefix} managementapp list`
 };

package/dist/m365/spo/commands/field/field-list.js

@@ -0,0 +1,84 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const request_1 = require("../../../../request");
+const utils_1 = require("../../../../utils");
+const SpoCommand_1 = require("../../../base/SpoCommand");
+const commands_1 = require("../../commands");
+class SpoFieldListCommand extends SpoCommand_1.default {
+    get name() {
+        return commands_1.default.FIELD_LIST;
+    }
+    get description() {
+        return 'Retrieves columns for the specified list or site';
+    }
+    getTelemetryProperties(args) {
+        const telemetryProps = super.getTelemetryProperties(args);
+        telemetryProps.listId = typeof args.options.listId !== 'undefined';
+        telemetryProps.listTitle = typeof args.options.listTitle !== 'undefined';
+        telemetryProps.listUrl = typeof args.options.listUrl !== 'undefined';
+        return telemetryProps;
+    }
+    defaultProperties() {
+        return ['Id', 'Title', 'Group', 'Hidden'];
+    }
+    commandAction(logger, args, cb) {
+        let listUrl = '';
+        if (args.options.listId) {
+            listUrl = `lists(guid'${encodeURIComponent(args.options.listId)}')/`;
+        }
+        else if (args.options.listTitle) {
+            listUrl = `lists/getByTitle('${encodeURIComponent(args.options.listTitle)}')/`;
+        }
+        else if (args.options.listUrl) {
+            const listServerRelativeUrl = utils_1.urlUtil.getServerRelativePath(args.options.webUrl, args.options.listUrl);
+            listUrl = `GetList('${encodeURIComponent(listServerRelativeUrl)}')/`;
+        }
+        const requestOptions = {
+            url: `${args.options.webUrl}/_api/web/${listUrl}fields`,
+            headers: {
+                accept: 'application/json;odata=nometadata'
+            },
+            responseType: 'json'
+        };
+        request_1.default
+            .get(requestOptions)
+            .then((res) => {
+            logger.log(res.value);
+            cb();
+        }, (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
+    }
+    options() {
+        const options = [
+            {
+                option: '-u, --webUrl <webUrl>'
+            },
+            {
+                option: '-t, --listTitle [listTitle]'
+            },
+            {
+                option: '-i, --listId [listId]'
+            },
+            {
+                option: '--listUrl [listUrl]'
+            }
+        ];
+        const parentOptions = super.options();
+        return options.concat(parentOptions);
+    }
+    validate(args) {
+        const isValidSharePointUrl = utils_1.validation.isValidSharePointUrl(args.options.webUrl);
+        if (isValidSharePointUrl !== true) {
+            return isValidSharePointUrl;
+        }
+        if (args.options.listId && !utils_1.validation.isValidGuid(args.options.listId)) {
+            return `${args.options.listId} is not a valid GUID`;
+        }
+        const listOptions = [args.options.listId, args.options.listTitle, args.options.listUrl];
+        if (listOptions.some(item => item !== undefined) && listOptions.filter(item => item !== undefined).length > 1) {
+            return `Specify either list id or title or list url`;
+        }
+        return true;
+    }
+}
+module.exports = new SpoFieldListCommand();
+//# sourceMappingURL=field-list.js.map

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

@@ -0,0 +1,84 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const request_1 = require("../../../../request");
+const utils_1 = require("../../../../utils");
+const SpoCommand_1 = require("../../../base/SpoCommand");
+const commands_1 = require("../../commands");
+class SpoListRoleInheritanceBreakCommand extends SpoCommand_1.default {
+    get name() {
+        return commands_1.default.LIST_ROLEINHERITANCE_BREAK;
+    }
+    get description() {
+        return 'Breaks role inheritance on list or library';
+    }
+    optionSets() {
+        return [
+            ['listId', 'listTitle']
+        ];
+    }
+    getTelemetryProperties(args) {
+        const telemetryProps = super.getTelemetryProperties(args);
+        telemetryProps.listId = typeof args.options.listId !== 'undefined';
+        telemetryProps.listTitle = typeof args.options.listTitle !== 'undefined';
+        telemetryProps.clearExistingPermissions = args.options.clearExistingPermissions === true;
+        return telemetryProps;
+    }
+    commandAction(logger, args, cb) {
+        if (this.verbose) {
+            logger.logToStderr(`Breaking role inheritance of list in site at ${args.options.webUrl}...`);
+        }
+        let requestUrl = `${args.options.webUrl}/_api/web/lists`;
+        if (args.options.listId) {
+            requestUrl += `(guid'${encodeURIComponent(args.options.listId)}')`;
+        }
+        else {
+            requestUrl += `/getbytitle('${encodeURIComponent(args.options.listTitle)}')`;
+        }
+        let keepExistingPermissions = true;
+        if (args.options.clearExistingPermissions) {
+            keepExistingPermissions = !args.options.clearExistingPermissions;
+        }
+        const requestOptions = {
+            url: `${requestUrl}/breakroleinheritance(${keepExistingPermissions})`,
+            method: 'POST',
+            headers: {
+                'accept': 'application/json;odata=nometadata',
+                'content-type': 'application/json'
+            },
+            responseType: 'json'
+        };
+        request_1.default
+            .post(requestOptions)
+            .then(_ => cb(), (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
+    }
+    options() {
+        const options = [
+            {
+                option: '-u, --webUrl <webUrl>'
+            },
+            {
+                option: '-i, --listId [listId]'
+            },
+            {
+                option: '-t, --listTitle [listTitle]'
+            },
+            {
+                option: '-c, --clearExistingPermissions'
+            }
+        ];
+        const parentOptions = super.options();
+        return options.concat(parentOptions);
+    }
+    validate(args) {
+        const isValidSharePointUrl = utils_1.validation.isValidSharePointUrl(args.options.webUrl);
+        if (isValidSharePointUrl !== true) {
+            return isValidSharePointUrl;
+        }
+        if (args.options.listId && !utils_1.validation.isValidGuid(args.options.listId)) {
+            return `${args.options.listId} is not a valid GUID`;
+        }
+        return true;
+    }
+}
+module.exports = new SpoListRoleInheritanceBreakCommand();
+//# sourceMappingURL=list-roleinheritance-break.js.map

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

@@ -0,0 +1,76 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const request_1 = require("../../../../request");
+const utils_1 = require("../../../../utils");
+const SpoCommand_1 = require("../../../base/SpoCommand");
+const commands_1 = require("../../commands");
+class SpoListRoleInheritanceResetCommand extends SpoCommand_1.default {
+    get name() {
+        return commands_1.default.LIST_ROLEINHERITANCE_RESET;
+    }
+    get description() {
+        return 'Restores role inheritance on list or library';
+    }
+    optionSets() {
+        return [
+            ['listId', 'listTitle']
+        ];
+    }
+    getTelemetryProperties(args) {
+        const telemetryProps = super.getTelemetryProperties(args);
+        telemetryProps.listId = typeof args.options.listId !== 'undefined';
+        telemetryProps.listTitle = typeof args.options.listTitle !== 'undefined';
+        return telemetryProps;
+    }
+    commandAction(logger, args, cb) {
+        if (this.verbose) {
+            logger.logToStderr(`Restore role inheritance of list in site at ${args.options.webUrl}...`);
+        }
+        let requestUrl = `${args.options.webUrl}/_api/web/lists`;
+        if (args.options.listId) {
+            requestUrl += `(guid'${encodeURIComponent(args.options.listId)}')`;
+        }
+        else {
+            requestUrl += `/getbytitle('${encodeURIComponent(args.options.listTitle)}')`;
+        }
+        const requestOptions = {
+            url: `${requestUrl}/resetroleinheritance`,
+            method: 'POST',
+            headers: {
+                'accept': 'application/json;odata=nometadata',
+                'content-type': 'application/json'
+            },
+            responseType: 'json'
+        };
+        request_1.default
+            .post(requestOptions)
+            .then(_ => cb(), (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
+    }
+    options() {
+        const options = [
+            {
+                option: '-u, --webUrl <webUrl>'
+            },
+            {
+                option: '-i, --listId [listId]'
+            },
+            {
+                option: '-t, --listTitle [listTitle]'
+            }
+        ];
+        const parentOptions = super.options();
+        return options.concat(parentOptions);
+    }
+    validate(args) {
+        const isValidSharePointUrl = utils_1.validation.isValidSharePointUrl(args.options.webUrl);
+        if (isValidSharePointUrl !== true) {
+            return isValidSharePointUrl;
+        }
+        if (args.options.listId && !utils_1.validation.isValidGuid(args.options.listId)) {
+            return `${args.options.listId} is not a valid GUID`;
+        }
+        return true;
+    }
+}
+module.exports = new SpoListRoleInheritanceResetCommand();
+//# sourceMappingURL=list-roleinheritance-reset.js.map

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

@@ -0,0 +1,83 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const request_1 = require("../../../../request");
+const utils_1 = require("../../../../utils");
+const SpoCommand_1 = require("../../../base/SpoCommand");
+const commands_1 = require("../../commands");
+class SpoListItemRoleInheritanceBreakCommand extends SpoCommand_1.default {
+    get name() {
+        return commands_1.default.LISTITEM_ROLEINHERITANCE_BREAK;
+    }
+    get description() {
+        return 'Break inheritance of list item';
+    }
+    optionSets() {
+        return [
+            ['listId', 'listTitle']
+        ];
+    }
+    commandAction(logger, args, cb) {
+        if (this.verbose) {
+            logger.logToStderr(`Breaking role inheritance of list item in site at ${args.options.webUrl}...`);
+        }
+        let requestUrl = `${args.options.webUrl}/_api/web/lists`;
+        if (args.options.listId) {
+            requestUrl += `(guid'${encodeURIComponent(args.options.listId)}')`;
+        }
+        else {
+            requestUrl += `/getbytitle('${encodeURIComponent(args.options.listTitle)}')`;
+        }
+        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'
+        };
+        request_1.default
+            .post(requestOptions)
+            .then(_ => cb(), (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
+    }
+    options() {
+        const options = [
+            {
+                option: '-u, --webUrl <webUrl>'
+            },
+            {
+                option: '--listItemId <listItemId>'
+            },
+            {
+                option: '-l, --listId [listId]'
+            },
+            {
+                option: '-t, --listTitle [listTitle]'
+            },
+            {
+                option: '-c, --clearExistingPermissions'
+            }
+        ];
+        const parentOptions = super.options();
+        return options.concat(parentOptions);
+    }
+    validate(args) {
+        const isValidSharePointUrl = utils_1.validation.isValidSharePointUrl(args.options.webUrl);
+        if (isValidSharePointUrl !== true) {
+            return isValidSharePointUrl;
+        }
+        if (args.options.listId && !utils_1.validation.isValidGuid(args.options.listId)) {
+            return `${args.options.listId} is not a valid GUID`;
+        }
+        if (isNaN(args.options.listItemId)) {
+            return `${args.options.listItemId} is not a number`;
+        }
+        return true;
+    }
+}
+module.exports = new SpoListItemRoleInheritanceBreakCommand();
+//# sourceMappingURL=listitem-roleinheritance-break.js.map

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

@@ -0,0 +1,79 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const request_1 = require("../../../../request");
+const utils_1 = require("../../../../utils");
+const SpoCommand_1 = require("../../../base/SpoCommand");
+const commands_1 = require("../../commands");
+class SpoListItemRoleInheritanceResetCommand extends SpoCommand_1.default {
+    get name() {
+        return commands_1.default.LISTITEM_ROLEINHERITANCE_RESET;
+    }
+    get description() {
+        return 'Restores the role inheritance of list item, file, or folder';
+    }
+    getTelemetryProperties(args) {
+        const telemetryProps = super.getTelemetryProperties(args);
+        telemetryProps.listId = typeof args.options.listId !== 'undefined';
+        telemetryProps.listTitle = typeof args.options.listTitle !== 'undefined';
+        return telemetryProps;
+    }
+    commandAction(logger, args, cb) {
+        let requestUrl = `${args.options.webUrl}/_api/web/lists`;
+        if (args.options.listId) {
+            requestUrl += `(guid'${encodeURIComponent(args.options.listId)}')`;
+        }
+        else {
+            requestUrl += `/getbytitle('${encodeURIComponent(args.options.listTitle)}')`;
+        }
+        const requestOptions = {
+            url: `${requestUrl}/items(${args.options.listItemId})/resetroleinheritance`,
+            method: 'POST',
+            headers: {
+                'accept': 'application/json;odata=nometadata',
+                'content-type': 'application/json'
+            },
+            responseType: 'json'
+        };
+        request_1.default
+            .post(requestOptions)
+            .then(_ => cb(), (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
+    }
+    optionSets() {
+        return [
+            ['listId', 'listTitle']
+        ];
+    }
+    options() {
+        const options = [
+            {
+                option: '-u, --webUrl <webUrl>'
+            },
+            {
+                option: '--listItemId <listItemId>'
+            },
+            {
+                option: '--listId [listId]'
+            },
+            {
+                option: '--listTitle [listTitle]'
+            }
+        ];
+        const parentOptions = super.options();
+        return options.concat(parentOptions);
+    }
+    validate(args) {
+        const isValidSharePointUrl = utils_1.validation.isValidSharePointUrl(args.options.webUrl);
+        if (isValidSharePointUrl !== true) {
+            return isValidSharePointUrl;
+        }
+        if (args.options.listId && !utils_1.validation.isValidGuid(args.options.listId)) {
+            return `${args.options.listId} is not a valid GUID`;
+        }
+        if (isNaN(args.options.listItemId)) {
+            return `${args.options.listItemId} is not a number`;
+        }
+        return true;
+    }
+}
+module.exports = new SpoListItemRoleInheritanceResetCommand();
+//# sourceMappingURL=listitem-roleinheritance-reset.js.map

package/dist/m365/spo/commands/roledefinition/roledefinition-list.js

@@ -0,0 +1,49 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const request_1 = require("../../../../request");
+const utils_1 = require("../../../../utils");
+const SpoCommand_1 = require("../../../base/SpoCommand");
+const commands_1 = require("../../commands");
+class SpoRoleDefinitionListCommand extends SpoCommand_1.default {
+    get name() {
+        return commands_1.default.ROLEDEFINITION_LIST;
+    }
+    get description() {
+        return 'Gets list of role definitions for the specified site';
+    }
+    defaultProperties() {
+        return ['Id', 'Name'];
+    }
+    commandAction(logger, args, cb) {
+        if (this.verbose) {
+            logger.logToStderr(`Getting role definitions list from ${args.options.webUrl}...`);
+        }
+        const requestOptions = {
+            url: `${args.options.webUrl}/_api/web/roledefinitions`,
+            headers: {
+                'accept': 'application/json;odata=nometadata'
+            },
+            responseType: 'json'
+        };
+        request_1.default
+            .get(requestOptions)
+            .then((response) => {
+            logger.log(response.value);
+            cb();
+        }, (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
+    }
+    options() {
+        const options = [
+            {
+                option: '-u, --webUrl <webUrl>'
+            }
+        ];
+        const parentOptions = super.options();
+        return options.concat(parentOptions);
+    }
+    validate(args) {
+        return utils_1.validation.isValidSharePointUrl(args.options.webUrl);
+    }
+}
+module.exports = new SpoRoleDefinitionListCommand();
+//# sourceMappingURL=roledefinition-list.js.map

package/dist/m365/spo/commands.js

@@ -41,6 +41,7 @@ exports.default = {
     FEATURE_LIST: `${prefix} feature list`,
     FIELD_ADD: `${prefix} field add`,
     FIELD_GET: `${prefix} field get`,
+    FIELD_LIST: `${prefix} field list`,
     FIELD_REMOVE: `${prefix} field remove`,
     FIELD_SET: `${prefix} field set`,
     FILE_ADD: `${prefix} file add`,
@@ -95,6 +96,8 @@ exports.default = {
     LIST_LABEL_SET: `${prefix} list label set`,
     LIST_LIST: `${prefix} list list`,
     LIST_REMOVE: `${prefix} list remove`,
+    LIST_ROLEINHERITANCE_BREAK: `${prefix} list roleinheritance break`,
+    LIST_ROLEINHERITANCE_RESET: `${prefix} list roleinheritance reset`,
     LIST_SET: `${prefix} list set`,
     LIST_SITESCRIPT_GET: `${prefix} list sitescript get`,
     LIST_VIEW_GET: `${prefix} list view get`,
@@ -117,6 +120,8 @@ exports.default = {
     LISTITEM_RECORD_DECLARE: `${prefix} listitem record declare`,
     LISTITEM_RECORD_UNDECLARE: `${prefix} listitem record undeclare`,
     LISTITEM_REMOVE: `${prefix} listitem remove`,
+    LISTITEM_ROLEINHERITANCE_BREAK: `${prefix} listitem roleinheritance break`,
+    LISTITEM_ROLEINHERITANCE_RESET: `${prefix} listitem roleinheritance reset`,
     LISTITEM_SET: `${prefix} listitem set`,
     MAIL_SEND: `${prefix} mail send`,
     NAVIGATION_NODE_ADD: `${prefix} navigation node add`,
@@ -159,6 +164,7 @@ exports.default = {
     REPORT_SITEUSAGEPAGES: `${prefix} report siteusagepages`,
     REPORT_SITEUSAGESITECOUNTS: `${prefix} report siteusagesitecounts`,
     REPORT_SITEUSAGESTORAGE: `${prefix} report siteusagestorage`,
+    ROLEDEFINITION_LIST: `${prefix} roledefinition list`,
     SEARCH: `${prefix} search`,
     SERVICEPRINCIPAL_GRANT_ADD: `${prefix} serviceprincipal grant add`,
     SERVICEPRINCIPAL_GRANT_LIST: `${prefix} serviceprincipal grant list`,

package/dist/m365/teams/commands/app/app-list.js

@@ -76,7 +76,7 @@ class TeamsAppListCommand extends GraphCommand_1.default {
     commandAction(logger, args, cb) {
         this
             .getEndpointUrl(args)
-            .then(endpoint => utils_1.odata.getAllItems(endpoint, logger))
+            .then(endpoint => utils_1.odata.getAllItems(endpoint))
             .then((items) => {
             if (args.options.teamId || args.options.teamName) {
                 items.forEach(t => {

package/dist/m365/teams/commands/channel/channel-list.js

@@ -53,7 +53,7 @@ class TeamsChannelListCommand extends GraphCommand_1.default {
             if (args.options.type) {
                 endpoint += `?$filter=membershipType eq '${args.options.type}'`;
             }
-            return utils_1.odata.getAllItems(endpoint, logger);
+            return utils_1.odata.getAllItems(endpoint);
         })
             .then((items) => {
             logger.log(items);

package/dist/m365/teams/commands/channel/channel-member-add.js

@@ -153,8 +153,11 @@ class TeamsChannelMemberAddCommand extends GraphCommand_1.default {
             if (!channelItem) {
                 return Promise.reject(`The specified channel '${args.options.channelName}' does not exist in the Microsoft Teams team with ID '${teamId}'`);
             }
+            if (channelItem.membershipType !== "private") {
+                return Promise.reject(`The specified channel is not a private channel`);
+            }
             return Promise.resolve(channelItem.id);
-        }, err => { return Promise.reject(err); });
+        });
     }
     getUserId(args) {
         if (args.options.userId) {

package/dist/m365/teams/commands/channel/channel-member-list.js

@@ -40,7 +40,7 @@ class TeamsChannelMemberListCommand extends GraphCommand_1.default {
         })
             .then((channelId) => {
             const endpoint = `${this.resource}/v1.0/teams/${this.teamId}/channels/${channelId}/members`;
-            return utils_1.odata.getAllItems(endpoint, logger);
+            return utils_1.odata.getAllItems(endpoint);
         })
             .then((memberships) => {
             if (args.options.role) {

package/dist/m365/teams/commands/channel/channel-member-remove.js

@@ -135,6 +135,9 @@ class TeamsChannelMemberRemoveCommand extends GraphCommand_1.default {
             if (!channelItem) {
                 return Promise.reject(`The specified channel does not exist in the Microsoft Teams team`);
             }
+            if (channelItem.membershipType !== "private") {
+                return Promise.reject(`The specified channel is not a private channel`);
+            }
             return Promise.resolve(channelItem.id);
         });
     }