Skip to content

fix(react): avoid polluting global XMLHttpRequest #14

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
lixl39505 merged 3 commits into from
Jul 26, 2025
Merged

fix(react): avoid polluting global XMLHttpRequest #14

lixl39505 merged 3 commits into from
Jul 26, 2025

Conversation

@lixl39505
Copy link

@lixl39505 lixl39505 commented Jul 26, 2025

关联 Issue

主要改动

  1. 修复了：防止污染全局 XMLHttpRequest 对象

界面截图

image

测试计划

  • Theming Editor 正常工作

检查清单

  • 本次变更新增了文件需要被包含在 npm 包的文件，且对应 package.json 的 files 字段包括了这些新增的文件
  • 本次变更需要更新文档，并且我更新了相关文档，如果还没更新文档，请新建文档更新的 Issue 并关联上来
  • 本次变更的兼容性测试覆盖了 Chrome
  • 本次变更的兼容性测试覆盖了 Safari
  • 本次变更的兼容性测试覆盖了 PC 端
  • 本次变更的兼容性测试覆盖了移动端【手机浏览器、钱包内置浏览器】

FireTable
FireTable approved these changes Jul 26, 2025
View reviewed changes
Copy link

@FireTable FireTable left a comment

Choose a reason for hiding this comment

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

LGTM

@lixl39505 lixl39505 merged commit 54eac5e into master Jul 26, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@FireTable FireTable FireTable approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lixl39505 @FireTable