Skip to content
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

enable dangerouslyAllowMutability in waitFor utils #1096

Closed

Conversation

csantos42
Copy link
Contributor

Summary:
Fixes #1074

Adds dangerouslyAllowMutability by default to the waitFor helpers to unblock users that need that prop enabled in the returned selectors.

Differential Revision: D29461447

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Jun 30, 2021
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D29461447

Summary:
Pull Request resolved: facebookexperimental#1091

todo

Differential Revision: D29449437

fbshipit-source-id: 21f7156ce74520080f882c7f4bdb034764754885
…tal#1096)

Summary:
Pull Request resolved: facebookexperimental#1096

Fixes facebookexperimental#1074

Adds `dangerouslyAllowMutability` by default to the waitFor helpers to unblock users that need that prop enabled in the returned selectors.

Differential Revision: D29461447

fbshipit-source-id: cde6c44299ac56c9bb0066b2c845f333820e62df
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D29461447

@csantos42 csantos42 force-pushed the export-D29461447 branch 2 times, most recently from aeaff97 to 14a1e0d Compare June 30, 2021 19:06
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in e717737.

@drarmstr drarmstr added the bug Something isn't working label Jul 1, 2021
@drarmstr drarmstr self-requested a review July 1, 2021 00:38
csantos42 pushed a commit to csantos42/Recoil that referenced this pull request Jul 29, 2021
Summary: Adds short description of change made in PR facebookexperimental#1096 which fixed issue facebookexperimental#1074

Differential Revision: D29980856

fbshipit-source-id: b46d7588125ada69175faff9b9a0b562f4c991c6
AlexGuz23 pushed a commit to AlexGuz23/Recoil that referenced this pull request Nov 3, 2022
Summary:
Pull Request resolved: facebookexperimental/Recoil#1096

Fixes facebookexperimental/Recoil#1074

Adds `dangerouslyAllowMutability` by default to the waitFor helpers to unblock users that need that prop enabled in the returned selectors.

Reviewed By: drarmstr

Differential Revision: D29461447

fbshipit-source-id: 176fe501f1e663ed3dbe024443646d5c51f8a033
snipershooter0701 pushed a commit to snipershooter0701/Recoil that referenced this pull request Mar 5, 2023
Summary:
Pull Request resolved: facebookexperimental/Recoil#1096

Fixes facebookexperimental/Recoil#1074

Adds `dangerouslyAllowMutability` by default to the waitFor helpers to unblock users that need that prop enabled in the returned selectors.

Reviewed By: drarmstr

Differential Revision: D29461447

fbshipit-source-id: 176fe501f1e663ed3dbe024443646d5c51f8a033
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

waitForAll and other wait helpers should never freeze values
4 participants