-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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 incorrect useCustomUnits import #32248
Conversation
Size Change: 0 B Total Size: 1.86 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ This fixes the issue, as described in the testing instructions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Tested according to testing instructions, works as advertised!
Description
In #31822 we moved
useCustomUnits
to a different package, and exported it as__experimentalUseCustomUnits
.There is one case where the hook is incorrectly imported without the
__experimental
part, which causes some crashes in the Site Editor (see testing instructions).This PR simply fixes the import adding the
__experimental
part.How has this been tested?
To reproduce the crashes:
Try again with this PR: you should be able to open the Global Style's Site Title section.
Types of changes
Bug fix (non-breaking change which fixes an issue)
Checklist:
*.native.js
files for terms that need renaming or removal).