Skip to content

Fix type error for consumers on @types/react v19#409

Merged
fatadel merged 1 commit into
firefox-devtools:masterfrom
fatadel:remove-reacttext
May 28, 2026
Merged

Fix type error for consumers on @types/react v19#409
fatadel merged 1 commit into
firefox-devtools:masterfrom
fatadel:remove-reacttext

Conversation

@fatadel
Copy link
Copy Markdown

@fatadel fatadel commented May 28, 2026

The fix is needed to be able to upgrade the profiler to React 19.

React.ReactText was removed from @types/react v19. It was an alias for string | number, so substitute the underlying types directly so the SubMenuProps.title declaration keeps compiling against both v18 and v19.

React.ReactText was removed from @types/react v19. It was an alias for
string | number, so substitute the underlying types directly so the
SubMenuProps.title declaration keeps compiling against both v18 and v19.
@netlify
Copy link
Copy Markdown

netlify Bot commented May 28, 2026

Deploy Preview for firefox-devtools-react-contextmenu ready!

Name Link
🔨 Latest commit 789a33b
🔍 Latest deploy log https://app.netlify.com/projects/firefox-devtools-react-contextmenu/deploys/6a181b183f195000084d69c9
😎 Deploy Preview https://deploy-preview-409--firefox-devtools-react-contextmenu.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@fatadel fatadel requested a review from canova May 28, 2026 10:38
Copy link
Copy Markdown
Member

@canova canova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fatadel fatadel merged commit dc55595 into firefox-devtools:master May 28, 2026
6 checks passed
@fatadel fatadel deleted the remove-reacttext branch May 28, 2026 11:22
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