Skip to content

[chore] move sanitizeFieldRestProps and sanitizeInputRestProps to ra-core#10874

Merged
djhi merged 1 commit intonextfrom
chore/move-sanitize-props
Jul 31, 2025
Merged

[chore] move sanitizeFieldRestProps and sanitizeInputRestProps to ra-core#10874
djhi merged 1 commit intonextfrom
chore/move-sanitize-props

Conversation

@jonathanarnault
Copy link
Contributor

Problem

sanitizeFieldRestProps and sanitizeInputRestProps are generic and can be resused in Shadcn Admin Kit components

Solution

Move them to RA core

How To Test

N/A

Additional Checks

  • The PR targets master for a bugfix or a documentation fix, or next for a feature
  • The PR includes unit tests (if not possible, describe why)
  • The PR includes one or several stories (if not possible, describe why)
  • The documentation is up to date

Also, please make sure to read the contributing guidelines.

@djhi djhi changed the title Chore: move sanitizeFieldRestProps and sanitizeInputRestProps to ra-core from ra-mui [chore] move sanitizeFieldRestProps and sanitizeInputRestProps to ra-core Jul 31, 2025
@djhi djhi modified the milestones: 5.10.1, 5.11.0 Jul 31, 2025
@djhi djhi merged commit 72dbd01 into next Jul 31, 2025
15 checks passed
@djhi djhi deleted the chore/move-sanitize-props branch July 31, 2025 14:25
@fzaninotto
Copy link
Member

I don't agree with this change. Props like className, headerClassName and others don't belong to a headless package. I think this should be reverted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants