fix(devtools-internal): fix `NODE_ENV` conditional

aliemir commented 4 months ago

PR Checklist

Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://refine.dev/docs/guides-concepts/contributing/#commit-convention

Bugs / Features

[x] Related issue(s) linked
[x] Tests for the changes have been added
[x] Docs have been added / updated
[x] Changesets have been added https://refine.dev/docs/guides-concepts/contributing/#creating-a-changeset

Changes

When using Refine with React Native, process.env.NODE_ENV !== "development" ? () => ({}) : () => {...} conditional in useQuerySubscription hook was causing a syntax error. This PR fixes the issue by explicitly returning an empty object on non-development environments.

Not sure why this happens but using parentheses to return an empty object throws syntax error on React Native environment.

RK-671

changeset-bot[bot] commented 4 months ago

🦋 Changeset detected

Latest commit: 4654353b90b6a09cbf65e41d4c1fa82553a2a3fa

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

This PR includes changesets to release 2 packages

| Name | Type | | ---------------------------- | ----- | | @refinedev/devtools-internal | Patch | | @refinedev/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

cypress[bot] commented 4 months ago

Passing run #11140 ↗︎

0

379

37

0

Details:

Merge 4654353b90b6a09cbf65e41d4c1fa82553a2a3fa into 3f4dea97e1059baeb5a269533541...
Project: refine	Commit: `2c7608b33a ℹ️`
Status: Passed	Duration: 21:50 💡
Started: May 27, 2024 8:36 AM	Ended: May 27, 2024 8:58 AM

refinedev / refine