Skip to content

Changes required for the React integration#3493

Merged
Janpot merged 4 commits into
mui:masterfrom
Janpot:changesdew
May 6, 2024
Merged

Changes required for the React integration#3493
Janpot merged 4 commits into
mui:masterfrom
Janpot:changesdew

Conversation

@Janpot

@Janpot Janpot commented May 3, 2024

Copy link
Copy Markdown
Member

Bringing some of the changes on #3125 to master

  • Make sure the top-level component can be imported server side (not rendered). And not fail on window not existing while initializing browser runtime. We can just error when trying to use the runtime instead.
  • Avoid serverside rendering the react router. It's client side only
  • Add interop for the Next.js image imports
  • More decoupling of Toolpad project from the function runtime

@Janpot Janpot added the internal Behind-the-scenes enhancement. Formerly called “core”. label May 3, 2024
@Janpot Janpot changed the title Bring some changes for the Next.js integration Changes required for the Next.js integration May 3, 2024
@Janpot Janpot marked this pull request as ready for review May 6, 2024 08:23
@Janpot Janpot requested a review from a team May 6, 2024 08:24
@Janpot Janpot changed the title Changes required for the Next.js integration Changes required for the React integration May 6, 2024
@Janpot Janpot enabled auto-merge (squash) May 6, 2024 08:44
@Janpot Janpot merged commit cee3089 into mui:master May 6, 2024
@Janpot Janpot deleted the changesdew branch May 6, 2024 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Behind-the-scenes enhancement. Formerly called “core”.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants