Skip to content

Conversation

@shubham1206agra
Copy link
Contributor

Reverts #701

@shubham1206agra shubham1206agra requested a review from a team as a code owner December 6, 2025 15:56
@melvin-bot melvin-bot bot requested review from mjasikowski and removed request for a team December 6, 2025 15:57
@shubham1206agra
Copy link
Contributor Author

@roryabraham @carlosmiceli @TMisiukiewicz We need to revert the PR as it is making the selector stale even though correct dependency was given. Example PR where you can test this is Expensify/App#74819

@roryabraham
Copy link
Contributor

Sorry @shubham1206agra, I don't understand. useLiveRef would have only made the selector a stable reference. How would removing useLiveRef make the selector stale if it's used directly as a prop? I'd love to discuss this further. Can you open an issue please?

@roryabraham
Copy link
Contributor

Actually, we can just discuss in Expensify/App#70296

@roryabraham
Copy link
Contributor

Let's close this (at least temporarily) so @mjasikowski's K2 remains clean while we discuss

@roryabraham roryabraham closed this Dec 9, 2025
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