Skip to content

Conversation

heath-freenome
Copy link
Member

@heath-freenome heath-freenome commented May 22, 2025

Reasons for making this change

With MUI 7 having full ESM support, our tsc-alias replacer support is no longer needed, and in fact breaks the way MUI 7 ESM support works

  • Updated .gitIgnore to globalize the *Replacer.js and *Replace.cjs names
  • Deleted the muiReplacer.ts file as it is no longer needed
  • In @rjsf/mui, removed the tsc-alias replacer support as follows:
    • Deleted the tsconfig.replacer.json
    • Updated the package.json to remove the compileReplacer script and updated the build:ts script not to use it
    • Updated the tsconfig.build.json to remove the muiReplacer reference
  • Updated the CHANGELOG.md accordingly

Checklist

  • I'm updating documentation
  • I'm adding or updating code
    • I've added and/or updated tests. I've run npx nx run-many --target=build --exclude=@rjsf/docs && npm run test:update to update snapshots, if needed.
    • I've updated docs if needed
    • I've updated the changelog with a description of the PR
  • I'm adding a new feature
    • I've updated the playground with an example use of the feature

With MUI 7 having full ESM support, our `tsc-alias` replacer support is no longer needed
- Updated `.gitIgnore` to globalize the `*Replacer.js` and `*Replace.cjs` names
- Deleted the `muiReplacer.ts` file as it is no longer needed
- In `@rjsf/mui`, removed the `tsc-alias` replacer support as follows:
  - Deleted the `tsconfig.replacer.json`
  - Updated the `package.json` to remove the `compileReplacer` script and updated the `build:ts` script not to use it
  - Updated the `tsconfig.build.json` to remove the `muiReplacer` reference
- Updated the `CHANGELOG.md` accordingly
@heath-freenome heath-freenome merged commit ffef96a into rjsf-team:main May 22, 2025
4 checks passed
@heath-freenome heath-freenome deleted the fix-mui-7-esm-support branch May 22, 2025 20:59
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.

2 participants