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 package exports #9310

Closed
wants to merge 1 commit into from
Closed

Fix package exports #9310

wants to merge 1 commit into from

Conversation

djhi
Copy link
Collaborator

@djhi djhi commented Sep 25, 2023

This fixes imports in frameworks such as Remix

@smeng9
Copy link
Contributor

smeng9 commented Sep 26, 2023

Continuing the discussion from #9309 (comment) here.

After reviewing changed files if we want to solve the module import/export test case for vitest in previous discussion we at least have to touch the package.json of ra-core to include an exports field.

If we are only targeting solving remix in this pull request it's another story, and I can open a different issue for vitest on react-admin issue list.

@djhi
Copy link
Collaborator Author

djhi commented Oct 3, 2023

Actually not needed with Remix

@djhi djhi closed this Oct 3, 2023
@djhi djhi deleted the fix-package-exports branch October 3, 2023 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFR Ready For Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants