Skip to content

Update breaking changes doc #523

Open
MehakBindra wants to merge 7 commits into
mainfrom
core/breaking-doc
Open

Update breaking changes doc #523
MehakBindra wants to merge 7 commits into
mainfrom
core/breaking-doc

Conversation

@MehakBindra
Copy link
Copy Markdown
Collaborator

@MehakBindra MehakBindra commented May 22, 2026

Update breaking changes doc for core to reflect current status

Copilot AI review requested due to automatic review settings May 22, 2026 20:37
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

Updates the breaking-changes reduction plan documentation for core/src/Microsoft.Teams.Apps by reorganizing the inventory into “Pending”, “Doc-only”, “Not migrated”, and “Implemented” sections, and by recording which compatibility gaps are already addressed in the new core library.

Changes:

  • Reorganized the breaking-changes inventory into status-based sections and updated several entries to “IMPLEMENTED” with the corresponding fix notes and file locations.
  • Added/updated sample migration difficulty assessment to reflect remaining blockers only.
  • Clarified several architectural/documentation-only items and consolidated “review later” gaps.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread core/docs/ReduceBreakingChangesPlan.md Outdated
Comment thread core/docs/ReduceBreakingChangesPlan.md
Comment thread core/docs/ReduceBreakingChangesPlan.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 22, 2026 21:12
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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

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

Comment thread core/docs/ReduceBreakingChangesPlan.md
Comment thread core/docs/ReduceBreakingChangesPlan.md
Comment thread core/docs/ReduceBreakingChangesPlan.md
@corinagum
Copy link
Copy Markdown
Collaborator

PR description?

@MehakBindra MehakBindra enabled auto-merge (squash) May 27, 2026 17:03
Copilot AI review requested due to automatic review settings May 27, 2026 17:42
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

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

### BC-20: Missing handler registration methods

---
**Decision: REVIEW LATER** — 18 handler methods exist in the old library but not in the new.
.AddOAuth("graph");
builder.AddTeams(appBuilder);
```

Comment thread core/docs/ReduceBreakingChangesPlan.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants