Skip to content

docs(updating): update v9 migration guide to add missing breaking changes - #4620

Open
brandyscarney wants to merge 8 commits into
major-9.0from
docs/v9-breaking-updates
Open

docs(updating): update v9 migration guide to add missing breaking changes#4620
brandyscarney wants to merge 8 commits into
major-9.0from
docs/v9-breaking-updates

Conversation

@brandyscarney

@brandyscarney brandyscarney commented Aug 6, 2026

Copy link
Copy Markdown
Member

This PR does the following:

  • Splits Vue & Vue Router to separate sections to match React & React Router
  • Reorganizes the components so they're in alphabetical order
  • Adds the breaking change sections for the following PRs:
  • Adds the missing breaking changes for Modal & Router Outlet

Preview

@vercel

vercel Bot commented Aug 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ionic-docs Ready Ready Preview Aug 13, 2026 10:23pm

Request Review

@brandyscarney
brandyscarney marked this pull request as ready for review August 6, 2026 20:44
@brandyscarney
brandyscarney requested a review from a team as a code owner August 6, 2026 20:44
@brandyscarney
brandyscarney requested review from ShaneK and removed request for a team August 6, 2026 20:44

@thetaPC thetaPC 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.

LGTM

@ShaneK ShaneK left a comment

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.

Looks good, but I noticed some issues. Mostly nits, some of them - like the vue router version specificity - should be fixed before merging though

Comment thread docs/updating/9-0.md Outdated
Comment thread docs/updating/9-0.md Outdated
Comment thread docs/updating/9-0.md Outdated
Comment thread docs/updating/9-0.md Outdated
brandyscarney and others added 3 commits August 11, 2026 15:54
Co-authored-by: Shane <shane@shanessite.net>
Co-authored-by: Shane <shane@shanessite.net>
Co-authored-by: Shane <shane@shanessite.net>

@thetaPC thetaPC 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.

Heads up: line 11's :::info Breaking Changes is already migrated to :::info[Breaking Changes] in the pending main sync, so if that lands first you'll hit a one-line conflict here. Take the bracketed form.

Comment thread docs/updating/9-0.md
Comment thread docs/updating/9-0.md
Comment thread docs/updating/9-0.md
Comment thread docs/updating/9-0.md
Comment thread docs/updating/9-0.md
Comment thread docs/updating/9-0.md
Comment thread docs/updating/9-0.md
Comment thread docs/updating/9-0.md

@thetaPC thetaPC 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.

LGTM

@ShaneK ShaneK left a comment

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.

Looks good to me!

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