Skip to content

New script sample to export all the channels from a team#2793

Closed
sudharsank wants to merge 3 commits intopnp:mainfrom
sudharsank:export-channels
Closed

New script sample to export all the channels from a team#2793
sudharsank wants to merge 3 commits intopnp:mainfrom
sudharsank:export-channels

Conversation

@sudharsank
Copy link
Copy Markdown
Contributor

Docs
Author Sudharsan Kesavanarayanan
Description Export all the channels information in a CSV file for the team's team
Keywords Microsoft Teams, Governance

@waldekmastykarz
Copy link
Copy Markdown
Member

Thank you! We'll review it shortly 👍

Copy link
Copy Markdown
Member

@waldekmastykarz waldekmastykarz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nicely done with a few minor changes I've done when merging the PR 👍

Comment thread docs/mkdocs.yml
- 'Deploy a Microsoft Teams personal app and add it to users': 'sample-scripts/teams/install-personal-app.md'
- 'Deploy Microsoft Teams app from Azure DevOps': 'sample-scripts/teams/deploy-teams-app.md'
- 'Export conversations from Microsoft Teams Channels': 'sample-scripts/teams/export-teams-conversations.md'
- 'Export all channels from Microsoft Teams Team': 'sample-scripts/teams/export-all-channels-teams.md'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When adding items to the Table of contents, let's sort them alphabetically so that our users can find them more easily.

$results | Export-Csv -Path "Channels.csv" -NoTypeInformation
Write-Host "Completed."
} else {
Write-Information "No channesl found!"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo in channels

[Parameter(Mandatory = $false)][string] $teamName
) {
if(!$teamID -and !$teamName) {
Write-Error "Either 'Team ID' or 'Team Name' is required"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's use the property names (teamID and teamName) to make it clearer which values we expect

@waldekmastykarz waldekmastykarz added pr-merged hacktoberfest-accepted Accept for hacktoberfest, will merge later labels Nov 2, 2021
@waldekmastykarz
Copy link
Copy Markdown
Member

Merged manually. Thank you! 👏

@waldekmastykarz waldekmastykarz added this to the v4.2 milestone Nov 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted Accept for hacktoberfest, will merge later pr-merged pr-sample

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants