fix: replace lodash imports from root with subpath imports

refinedev / refine

A React Framework for building internal tools, admin panels, dashboards & B2B apps with unmatched flexibility.

https://refine.dev

MIT License

25.92k stars 1.96k forks source link

fix: replace lodash imports from root with subpath imports #6052

Closed aliemir closed 1 week ago

aliemir commented 2 weeks 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

ESM bundle of @refinedev/hasura was broken due to incorrect lodash import. Import has been replaced with subdirectory import to get handled properly in the bundling process.

Fixes #6044

@refinedev/devtools-server was using lodash imports from root which are interpreted as CJS imports in the ESM bundle. To avoid any future issues, lodash imports have been replaced with subdirectory imports.

ESM bundle of @refinedev/inferencer was broken due to incorrect lodash import. Import has been replaced with subdirectory import to get handled properly in the bundling process.

changeset-bot[bot] commented 2 weeks ago

🦋 Changeset detected

Latest commit: c6a15b2131575bfcdd55472b04f9c63862b556f5

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

This PR includes changesets to release 5 packages

| Name | Type | | -------------------------- | ----- | | @refinedev/hasura | Patch | | @refinedev/devtools-server | Patch | | @refinedev/inferencer | Patch | | @refinedev/cli | Patch | | @refinedev/devtools | 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 2 weeks ago

3 failed tests on run #11205 ↗︎

378

Details:

Merge c6a15b2131575bfcdd55472b04f9c63862b556f5 into 24db047aea42e307a9662c46fde5...
Project: refine	Commit: `b841ec8a14 ℹ️`
Status: Failed	Duration: 22:07 💡
Started: Jun 20, 2024 9:04 AM	Ended: Jun 20, 2024 9:26 AM

cypress/e2e/all.cy.ts • 1 failed test • table-mantine-advanced

View Output

Test		Artifacts
table-mantine-advanced > should fill the form with the row data when click the edit button and save the form		`Test Replay` `Screenshots`

cypress/e2e/all.cy.ts • 2 failed tests • form-mantine-use-form