Skip to content

Conversation

onurtemizkan
Copy link
Collaborator

@onurtemizkan onurtemizkan commented Dec 31, 2024

Resolves: #14848
Resolves: #14877

I believe this PR should resolve the issues above, but I am still unable to reproduce them.
We can follow up this PR with new tests when we get to reproduce.

@onurtemizkan onurtemizkan force-pushed the onur/react-router-recursion-break branch from 80b9537 to b925c14 Compare December 31, 2024 16:04
@onurtemizkan onurtemizkan marked this pull request as ready for review January 1, 2025 10:47
Copy link
Member

@s1gr1d s1gr1d left a comment

Choose a reason for hiding this comment

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

Thanks for optimizing. I have one little suggestion to use Array.from() instead of spreading as it's faster.

@chargome chargome merged commit 4e6c7cb into develop Jan 2, 2025
152 checks passed
@chargome chargome deleted the onur/react-router-recursion-break branch January 2, 2025 11:56
chargome pushed a commit that referenced this pull request Jan 2, 2025
@Bartozzz
Copy link

Bartozzz commented Jan 7, 2025

Nice! Could we get this released by any chance? 🙏🏻

chargome added a commit that referenced this pull request Jan 7, 2025
@chargome
Copy link
Member

chargome commented Jan 7, 2025

@Bartozzz we'll cut a release today

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

Labels

None yet

Projects

None yet

4 participants