Skip to content

docs: add note about read-only group calendar container properties#9880

Merged
Lauragra merged 3 commits into
microsoftgraph:mainfrom
mashokMicrosoft:mashok/group-calendar-readonly-note
May 4, 2026
Merged

docs: add note about read-only group calendar container properties#9880
Lauragra merged 3 commits into
microsoftgraph:mainfrom
mashokMicrosoft:mashok/group-calendar-readonly-note

Conversation

@mashokMicrosoft
Copy link
Copy Markdown

@mashokMicrosoft mashokMicrosoft commented Apr 27, 2026

Summary

  • Added an [!IMPORTANT] callout in the Request body section of calendar-update.md to clarify that calendar container properties in group calendars are read-only.
  • PATCH requests targeting these properties return 405 Method Not Allowed.

Test plan

  • Verify the callout renders correctly in the docs preview.
  • Confirm placement is after the request body property table and before the Response section.

🤖 Generated with Claude Code

…perties

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 7eb7a0f:

✅ Validation status: passed

File Status Preview URL Details
api-reference/v1.0/api/calendar-update.md ✅Succeeded

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 7eb7a0f:

✅ Validation status: passed

File Status Preview URL Details
api-reference/v1.0/api/calendar-update.md ✅Succeeded

For more details, please refer to the build report.

@Lauragra Lauragra self-assigned this May 4, 2026
Lauragra added 2 commits May 4, 2026 07:55
Added important note about group calendar properties being read-only.
@Lauragra Lauragra requested a review from Copilot May 4, 2026 14:56
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new warning to the calendar update API docs to clarify group-calendar update behavior for calendar container properties in both v1.0 and beta. This fits the Outlook/Graph reference set by documenting a group-specific API constraint on an existing PATCH method.

Changes:

  • Added an [!IMPORTANT] callout to the v1.0 calendar-update topic.
  • Added the same [!IMPORTANT] callout to the beta calendar-update topic.
  • Documented that certain PATCH requests against group calendar container properties return 405 Method Not Allowed.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
api-reference/v1.0/api/calendar-update.md Adds the new group-calendar warning to the v1.0 update method doc.
api-reference/beta/api/calendar-update.md Adds the same warning to the beta update method doc.

Comment thread api-reference/v1.0/api/calendar-update.md
Comment thread api-reference/beta/api/calendar-update.md
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 1df5a0d:

✅ Validation status: passed

File Status Preview URL Details
api-reference/v1.0/api/calendar-update.md ✅Succeeded

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 1d50f66:

✅ Validation status: passed

File Status Preview URL Details
api-reference/beta/api/calendar-update.md ✅Succeeded
api-reference/v1.0/api/calendar-update.md ✅Succeeded

For more details, please refer to the build report.

@Danipocket
Copy link
Copy Markdown
Contributor

@mashokMicrosoft, please do the CL agreement as indicated above.

@Lauragra Lauragra added the ready to merge Use this label to indicate to the repository admins that your PR is ready to merge into master. label May 4, 2026
@mashokMicrosoft
Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree company="Microsoft"

@Lauragra Lauragra merged commit e8f3497 into microsoftgraph:main May 4, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Use this label to indicate to the repository admins that your PR is ready to merge into master.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants