add missing migration note for isEditor to NodeApi.isEditor

[joshgillies]

Checklist

• yarn typecheck
• yarn lint:fix
• yarn test
• yarn brl
• yarn changeset
• ui changelog

[codesandbox]

Review or Edit in CodeSandbox

Open the branch in Web Editor • VS Code • Insiders

Open Preview

[changeset-bot]

⚠️ No Changeset found

Latest commit: 2556313

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
plate	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 22, 2025 8:49am

[joshgillies]

I note the internal API for isEditor was removed here #3952 in favor of the NodeApi.isEditor method. However during a recent migration from @udecode/plate@40 I found the migration guide misleading due to the following: `

plate/BREAKING_CHANGES.md

Line 426 in e913541

- `isEditor` -> `editor.api.isEditor`

This PR aims to avoid confusion for anyone else doing the upgrade.

[zbeyens]

Good catch, could you instead update the changelog / BREAKING_CHANGES.md? There is no added value to make it overridable in editor.api.

[joshgillies]

You mean change the BREAKING_CHANGES.md to:

- - `isEditor` -> `editor.api.isEditor` 
+ - `isEditor` -> `NodeApi.isEditor` 

[joshgillies]

@zbeyens have update the breaking changes to reflect the updates. Let me know if there's anything else

[zbeyens]

Thanks! The changeset can be removed.

[joshgillies]

No problem, thanks 😄