Skip to content

cleanup teams docs - #1424

Open
eschultink wants to merge 1 commit into
rc-v0.7.1from
s233-teams-docs
Open

cleanup teams docs#1424
eschultink wants to merge 1 commit into
rc-v0.7.1from
s233-teams-docs

Conversation

@eschultink

Copy link
Copy Markdown
Member

Description

  • Clarify Teams online meeting access setup.
  • Clean up connector documentation for better readability and understanding.

Change implications

  • dependencies added/changed? no
  • something important to note in future release notes? no

Call out the extra Entra/Teams policy step from Required Scopes, add the missing artifact scope, and fix copy in the PowerShell setup instructions.

Co-authored-by: Cursor <cursoragent@cursor.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The shared Microsoft 365 scopes table must include the newly required permission.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Updates Microsoft Teams documentation for online meeting permissions and access-policy setup.

Changes:

  • Adds OnlineMeetingArtifact.Read.All guidance.
  • Clarifies PowerShell instructions and corrects permission links.
File summaries
File Summary Review note
docs/sources/microsoft-365/msft-teams/README.md Documents Teams online meeting access requirements. nit (3 votes): Update the shared Teams scopes table with OnlineMeetingArtifact.Read.All.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

- [`ChannelMessage.Read.All`](https://learn.microsoft.com/en-us/graph/permissions-reference#channelmessagereadall)
- [`CallRecords.Read.All`](https://learn.microsoft.com/en-us/graph/permissions-reference#callrecordsreadall)
- [`OnlineMeetings.Read.All`](https://learn.microsoft.com/en-us/graph/permissions-reference#onlinemeetingsreadall)
- [`OnlineMeetingArtifact.Read.All`](https://learn.microsoft.com/en-us/graph/permissions-reference#onlinemeetingartifactreadall)

@aperez-worklytics aperez-worklytics left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

OK, but I've created #1423 with more instructions before this branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants