Skip to content
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.

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

fix(adapter): support objectKeys #1573

Merged
merged 3 commits into from
Jul 11, 2024
Merged

Conversation

fu050409
Copy link
Member

Description:

  • When merging configs, isPlainObject will judge the server object proxy and prevent by getter. So allow objectKeys for getter.

Fixed: #1568

This PR fixes a problem where an error is thrown after modifying farm.config.ts. In fact, the config was still unable to reload completely correctly. See #1572.

fu050409 added 2 commits July 10, 2024 11:15
- When merging configs, `isPlainObject` will judge the server object proxy and prevent by getter. So allow objectKeys for getter.
Copy link

changeset-bot bot commented Jul 10, 2024

🦋 Changeset detected

Latest commit: fc8f545

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@farmfe/core Patch

Not sure what this means? Click here to learn what changesets are.

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

@fu050409 fu050409 requested a review from wre232114 July 11, 2024 08:04
@wre232114 wre232114 enabled auto-merge (squash) July 11, 2024 11:34
@wre232114 wre232114 merged commit 7b0c4ff into farm-fe:main Jul 11, 2024
32 checks passed
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.

[Bug Report]: vue3项目,开发服务器开启状态下,修改了farm.config.ts,终端会报错
2 participants