Fix error handling in Remove-PnPTeamsChannel #4036
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fix Remove-PnPTeamsChannel error handling mismatch between "Team not found", "Channel not found" and no error.
|
I recommend viewing the diff with "Hide whitespace" enabled, as that shows that only a |
gautamdsheth
approved these changes
Jun 20, 2024
|
Yikes, good catch !! |
KoenZomers
pushed a commit
to KoenZomers/pnppowershell
that referenced
this pull request
Jun 24, 2024
Fix Remove-PnPTeamsChannel error handling mismatch between "Team not found", "Channel not found" and no error.
gautamdsheth
added a commit
that referenced
this pull request
Jun 27, 2024
* Minor documentation fixes (#3849) * Update Set-PnPTemporarilyDisableAppBar.md * Update Set-PnPTeamsTeamPicture.md * Update Set-PnPTeamsTeamArchivedState.md * Update Set-PnPTeamsTag.md --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Nightly publish to PowerShell Gallery * Update CHANGELOG.md * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * new cmdlet unlock-PnPSensitivityLabelEncryptedFile * Add ErrorMessage in batch delete progress * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Fix Copy-PnPList handling of lookup columns (#3870) Fix Copy-PnPList copying the list connected to the first lookup column instead of the specified list. Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Minor documentation fixes (#3853) * Added connection parameter description * Added connection parameter description * Update Set-PnPStructuralNavigationCacheWebState.md * Update Set-PnPStructuralNavigationCacheWebState.md * Update Set-PnPStructuralNavigationCacheWebState.md * Update Set-PnPStructuralNavigationCacheSiteState.md * Removed WhatIf parameter description * Update Set-PnPSiteVersionPolicy.md --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * add example (#3869) Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Additional settings for SetTenantSite.cs : LoopDefaultSharingLinkRole, DefaultShareLinkScope, DefaultShareLinkRole, LoopDefaultSharingLinkScope (#3874) * New parameters added to Set tenant site * updae * Update to add additional properties * updated files --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Adding upload entire local folder to SharePoint Online into `Copy-PnPFolder` (#3850) * Adding functionality to allow a local folder with all its files and optionally recursed subfolders to be uploaded to SharePoint Online * Added PR reference * Typo fix * Adding verbose parameter * Fixing syntax issue * Added that empty folders will also be removed when providing -RemoveAfterCopy * Updated help text to reflect folders being deleted now as well --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Renaming Progress to Status as discussed * Added changelog entry * Adding changelog entry * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Fix exception in PnPConnectedCmdlet (#3885) Fix an NullDereferenceException happening when an exception is logged in PnPConnectedCmdlet but the connection passed through -Connection parameter is not the latest one. Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Minor Doc change: Add-PnPFolderUserSharingLink (#3881) Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Adding support for vanity domain tenants in `Get-PnPSiteCollectionAppCatalog` and `Get-PnPTenantSite` (#3895) * Adding support for vanity domain tenants * Added changelog entry * Grammar fix --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Fix NullDereferenceException in Get-PnPUserProfileProperty (#3891) Fix NullDereferenceException happening in Get-PnPUserProfileProperty when the user profile doesn't exist, showing a better error message Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * replace ParentTerm by ParentTermId (#3890) Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Nightly publish to PowerShell Gallery * Adding support for vanity domains to several cmdlets (#3898) * Updating cmdlets to use the TenantAdminUrl if possible before trying to guess the tenant admin URL * Adding PR reference * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Preparing for Managed Identity support in PnP Framework (#3857) Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Fix destination folder definition in build process on Mac dev machines (#3907) * fix destination folder definition * fix destination folder definition --------- Co-authored-by: Tobias Maestrini <tobias@bee365.ch> * Update CHANGELOG.md * Minor documentation fixes (#3893) * Update Set-PnPSiteTemplateMetadata.md * Added Connection parameter description * Update Set-PnPSiteGroup.md * Update Set-PnPSiteDesign.md * Update Set-PnPSearchSettings.md * Update Set-PnPRoleDefinition.md * Update Set-PnPRetentionLabel.md * Removed common parameters * Added connection parameter description * Removed common parameters * Update Set-PnPPageWebPart.md * Removed [] from mandatory parameters --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Feature: bump MSAL.NET to latest version (#3905) Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> * Nightly publish to PowerShell Gallery * Bump alpine linux version * Update release.yml * Create codeql.yml (#3915) * Update codeql.yml * Update codeql.yml * Update codeql.yml * Update codeql.yml * Update codeql.yml * Update codeql.yml * Update buildpr.yml * Nightly publish to PowerShell Gallery * Update hub.docker.md * Update pnppowershell.dockerFile * Update Publish-UnpublishedImage.ps1 * Update README.md * Update dev-containers.md * Update dev-containers.md * Update docker.md * Create scorecard.yml * Update scorecard.yml * Update scorecard.yml * Update scorecard.yml * Update scorecard.yml * Update scorecard.yml * Update scorecard.yml * Update scorecard.yml * Update README.md * Nightly publish to PowerShell Gallery * Update codeql.yml * Update closestaleissues.yml * Update cleanupnightlyreleases.yml * Update buildpr.yml * Update buildexternalhelp.yml * Update buildpr.yml * Update -Schema parameter documentation (#3923) Add missing Schema versions V202103 and V202209 to all -Schema parameters in different cmdlets Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update buildpr.yml * Update builddocsite.yml * Update checkdocumentationbuild.yml * Update nightlyrelease.yml * Update Publish-UnpublishedImage.ps1 * Update pnppowershell.dockerFile * Update Publish-UnpublishedImage.ps1 * Update release.yml * Update nightlyrelease.yml * Update nightlyrelease.yml * Update release.yml * Update LICENSE * Update shared.ps1 * Delete samples/Provisioning.SelfHostedWithAzureWebJob/Engine/nuget.exe * Update codeql.yml * Create SECURITY.md * Update codeql.yml * Update README.md * Nightly publish to PowerShell Gallery * Update README.md * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Updated spo-tenant with additional tenant settings (#3930) * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Update builddocsite.yml * Added Exchange Online properties of Microsoft 365 Groups (#3958) * Added implementation * Added PR reference * Fixing documentation build issue --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Allow to pass Group parameter to Remove-PnPGroupMember from pipeline (#3955) Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Adds documentation for missing parameter MarkNewFilesSensitiveByDefault to set-spotenant (#3936) * Updated spo-tenant with additional tenant settings * Update doc to include MarkNewFilesSensitiveByDefault * remove a parameter --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update SiteIDtoURL.ps1 MFA, Script Execution * Update version.txt * Nightly publish to PowerShell Gallery * Update builddocsite.yml * closes #3945 (#3947) Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Fixed some grammar issues (#3960) * Fix grammar issues * added to changelog * reviewer comments --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Bump MSAL.NET to latest * Change runner back to linux * Amend to return additional parameters (#3948) Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * New cmdlet for getcontainerType and minor changes to docs (#3946) * Update CHANGELOG.md * Allow to pass empty values to Set-PnPTenantCdnPolicy (#3937) Change Set-PnPTenantCdnPolicy to allow PolicyValue to be an empty string or $null, while still being mandatory. Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Update Disable-PnPTenantServicePrincipal.md (#3975) Instead of enabling this cmdlet should disable the PnPTenantServicePrincipal Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Add additional settings RecycleBinRetentionPeriod,OneDriveBlockGuestsAsSiteAdmin,OneDriveDefaultShareLinkRole,OneDriveDefaultShareLinkScope,OneDriveDefaultLinkToExistingAccess (#3977) Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Improve security * Nightly publish to PowerShell Gallery * Files for new cmdlets Set-PnPTenantRestrictedSearchMode and GetTenantRestrictedSearchMode.cs (#3976) * Files for rss * update to executequeryretry * add client --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Updated for MacOS * Additional tenant settings from api/SPOInternalUseOnly.TenantAdminSettings (#3902) * internal settings files * Added additional clarification * update get-PnPTenantInternalSettings to get-PnPTenantInternalSetting * Update to use "System.Text.Json;" and remove other references. --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Nightly publish to PowerShell Gallery * Remove obsolete UserVoiceForFeedbackEnabled parameter in SetTenant.cs (#3985) Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> * Update CHANGELOG.md * chore: Update Set-PnPHomeSite.md and SetHomeSite.cs (#3986) Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> * Update PnPTenantSite to include additional properties and their setters (#3987) Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> * Update CHANGELOG.md * feat: Add ExecuteQueryRetry to Admin commands for improved reliability (#3988) Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> * Nightly publish to PowerShell Gallery * cmdlet to add home site (#3989) Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Added new cmdlet to return modern page scheduling enabled status * Moved cmdlets to Pages namespace * Update CHANGELOG.md * feat: Add SortByStorage parameter to GetContainer cmdlet & fix pagination (#3990) Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> * Update CHANGELOG.md * Nightly publish to PowerShell Gallery * feat: Include sensitivity labels in Get-PnPMicrosoft365Group command (#3991) * feat: Update Microsoft365Group commands to include IncludeSensitivityLabels parameter * feat: Include sensitivity labels in Get-PnPMicrosoft365Group command --------- Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> * Update CHANGELOG.md * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Updated contributors * Remove uservoice property since it is deprecated * Update CHANGELOG.md * Trying macos runner * Update buildpr.yml * Fix New-PnPTeamsTeam sometimes "Conflict (409): Team already exists" (#3992) Handle "Conflict (409): Team already exists" error as a case of "a previous teamify call succeeded". Fix #3964 Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Feature: added new cmdlet Get-PnPFileSensitivityLabelInfo (#3994) * Feature: added new cmdlet Get-PnPFileSensitivityLabelInfo * Update SPOFileSensitivityLabelInfo.cs --------- Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> * Update CHANGELOG.md * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * chore: Update Get-PnPHomeSite to include Viva Connections functionality (#4002) Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Updated permissions according to findings shared in #277 * Nightly publish to PowerShell Gallery * Removing faulty brackets * Fixing update done through #3958 (#4013) * Adding additional information, removing alias as it works inversed from the original cmdlet name, only kept the Graph name and ditched the Exchange Online name * Added code to distinguish the type of token being used (Delegate vs AppOnly) and using it to show a warning if properties are being set with the wrong token type * Renaming method as this makes more sense --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update SetSiteVersionPolicy.cs (#4005) Changed "Progress" to "Status" as the cmdlet has been renamed as well. Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Security: Bump MSAL.NET to latest (#4018) * Nightly publish to PowerShell Gallery * Updated synopsis for Graph API limitations Added a comment in the Synopsis section for Graph API limitations where some `-Type` values are not configurable (PowerBI). * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Updated documentation for Set-PnPFolderPermission (#4025) Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update Set-PnPList.md (#4020) See #3783 where an issue is described when this feature is not activated. So maybe this PR makes the documentation more clear. Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * New cmdlet for Get-PnPTenantRestrictedSearchAllowedList (#3997) * new cmdlet * correct space * add documentation --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Feature #4028 : return EnableAIPIntegation in Get-PnPTenant cmdlet (#4030) Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> * Fix #3908: improve connection validation in ConnectOnline.cs (#4031) Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> * Adding `-IsSharePointAddInsDisabled` to `Set-PnPTenant` (#4032) * Added -IsSharePointAddInsDisabled to Set-PnPTenant * Added PR reference * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Fix error handling in Remove-PnPTeamsChannel (#4036) Fix Remove-PnPTeamsChannel error handling mismatch between "Team not found", "Channel not found" and no error. * Update CHANGELOG.md * More settings related to version trim ,ReadOnlyForUnmanagedDevices and RestrictContentOrgWideSearch (#4024) * More settings related to version trim ,ReadOnlyForUnmanagedDevices and RestrictContentOrgWideSearch * remove reference to forms * remove version trim settings --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Update Copy-PnPFile.md Removed the comment saying we couldn't copy files between SPO and ODFB. We've created this functionality in previous versions (see issue #1720). * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Fix #4041 : issue retrieving loop sharing information (#4047) Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> * Update AddListFoldersToSiteTemplate.cs (#3918) Co-authored-by: Maxime HAZEBROUCQ (Ext) <maxime.hazebroucq@ext.hermes.com> Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Nightly publish to PowerShell Gallery * Bump references for 2.5 release --------- Co-authored-by: Arleta Wanat <42035526+PowershellScripts@users.noreply.github.com> Co-authored-by: erwinvanhunen <erwinvanhunen@users.noreply.github.com> Co-authored-by: reshmee011 <reshmee011@gmail.com> Co-authored-by: jennywu <jennywu@microsoft.com_odspmdb> Co-authored-by: Giacomo Pozzoni <giacomopoz@gmail.com> Co-authored-by: Aimery Thomas <7100077+a1mery@users.noreply.github.com> Co-authored-by: Koen Zomers <koen@zomers.eu> Co-authored-by: WCONFR <68463689+WCONFR@users.noreply.github.com> Co-authored-by: Tobias Maestrini <69770609+tmaestrini@users.noreply.github.com> Co-authored-by: Tobias Maestrini <tobias@bee365.ch> Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> Co-authored-by: gautamdsheth <gautamdsheth@users.noreply.github.com> Co-authored-by: Sam Larson (Microsoft) <salarson@microsoft.com> Co-authored-by: Christian Veenhuis <124370897+ChVeen@users.noreply.github.com> Co-authored-by: Mark Gort <52573368+markgort86@users.noreply.github.com> Co-authored-by: Erwin van Hunen <erwin.van.hunen@outlook.com> Co-authored-by: Marc Studer <31045631+Studermarc@users.noreply.github.com> Co-authored-by: Veronique Lengelle <25181757+veronicageek@users.noreply.github.com> Co-authored-by: Paolo Pialorsi <paolo@pialorsi.com> Co-authored-by: mhazebroucq <mhazebroucq@gmail.com> Co-authored-by: Maxime HAZEBROUCQ (Ext) <maxime.hazebroucq@ext.hermes.com>
gautamdsheth
added a commit
that referenced
this pull request
Jun 30, 2024
* Adding additional information, removing alias as it works inversed from the original cmdlet name, only kept the Graph name and ditched the Exchange Online name * Added code to distinguish the type of token being used (Delegate vs AppOnly) and using it to show a warning if properties are being set with the wrong token type * Renaming method as this makes more sense * Changed code to pass along the cmdlet instance so logging can take place in deeper levels * Rewrote async functionality to become sync as PowerShell doesn't support async handling anyway * Optimized logging * Further fixes and optimization * Removing logging oauth token to verbose output * Add verbose logging of HTTP body * Removed async from method as it was no longer async anyway * Fixing Sync HttpClient Sends to support HTTP/2 * Further testing and optimization. Now also writing verbose if the permissions set through the attribute above the cmdlet are present in the current access token. * Added code to distinguish the type of token being used (Delegate vs AppOnly) and using it to show a warning if properties are being set with the wrong token type * Renaming method as this makes more sense * Removing logging oauth token to verbose output * Add verbose logging of HTTP body * Removed async from method as it was no longer async anyway * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Updated documentation for Set-PnPFolderPermission (#4025) Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update Set-PnPList.md (#4020) See #3783 where an issue is described when this feature is not activated. So maybe this PR makes the documentation more clear. Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * New cmdlet for Get-PnPTenantRestrictedSearchAllowedList (#3997) * new cmdlet * correct space * add documentation --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Feature #4028 : return EnableAIPIntegation in Get-PnPTenant cmdlet (#4030) Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> * Fix #3908: improve connection validation in ConnectOnline.cs (#4031) Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> * Adding `-IsSharePointAddInsDisabled` to `Set-PnPTenant` (#4032) * Added -IsSharePointAddInsDisabled to Set-PnPTenant * Added PR reference * Nightly publish to PowerShell Gallery * Nightly publish to PowerShell Gallery * Fix error handling in Remove-PnPTeamsChannel (#4036) Fix Remove-PnPTeamsChannel error handling mismatch between "Team not found", "Channel not found" and no error. * Update CHANGELOG.md * More settings related to version trim ,ReadOnlyForUnmanagedDevices and RestrictContentOrgWideSearch (#4024) * More settings related to version trim ,ReadOnlyForUnmanagedDevices and RestrictContentOrgWideSearch * remove reference to forms * remove version trim settings --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> * Update CHANGELOG.md * Fixing Sync HttpClient Sends to support HTTP/2 * Further testing and optimization. Now also writing verbose if the permissions set through the attribute above the cmdlet are present in the current access token. * Rebasing to close the gap with dev * Added changelog entry * Optimizations * Code cleanup and simplification * Fixing typo * Moving changelog entry to nightly --------- Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com> Co-authored-by: erwinvanhunen <erwinvanhunen@users.noreply.github.com> Co-authored-by: Paolo Pialorsi <paolo@pialorsi.com> Co-authored-by: Mark Gort <52573368+markgort86@users.noreply.github.com> Co-authored-by: Reshmee Auckloo <reshmee011@gmail.com> Co-authored-by: Gautam Sheth <gautam.sheth@staffbase.com> Co-authored-by: Giacomo Pozzoni <giacomopoz@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Type
Related Issues?
Fixes #4035
What is in this Pull Request ?
Fix Remove-PnPTeamsChannel error handling mismatch between "Team not found", "Channel not found" and no error.
A badly positioned
}was:Before:
After:
