Skip to content

Conversation

@HellowVirgil
Copy link
Collaborator

Description

Please include a summary of the changes and the motivation behind them. Also, include any relevant context or links to related issues.

Type of Change

Please mark the type of change:

  • Bug fix
  • New feature
  • Documentation update
  • Code refactoring
  • Other (please describe):

Related Issues

Please list any issues related to this pull request (e.g., Fixes #123, Closes #456).

Testing

Please describe the tests that were performed to verify your changes. Include details about the testing framework, if applicable.

  • Unit tests
  • Integration tests
  • Manual testing

Checklist

  • My code follows the project’s coding style.
  • I have updated the documentation as necessary.
  • I have added tests to cover my changes.
  • All new and existing tests pass.
  • My changes require a change to the documentation.

Additional Notes

如果组件被挂载(mount)后立即被卸载(unmount),或者在 getOrCreateEditorState 异步操作完成之前触发了 onViewUnmount ,此时 this.editor 尚未被赋值,仍然是 undefined ,导致报错。

@changeset-bot
Copy link

changeset-bot bot commented Dec 16, 2025

⚠️ No Changeset found

Latest commit: f832a38

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

@HellowVirgil HellowVirgil merged commit ec4bcc0 into main Dec 16, 2025
2 of 7 checks passed
@HellowVirgil HellowVirgil deleted the fix/editor-instance-is-undefined branch December 16, 2025 11:32
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