This repository has been archived by the owner on Jan 1, 2025. It is now read-only.
Make default optional for atoms #1639
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
facebook-github-bot
added
CLA Signed
|
This pull request was exported from Phabricator. Differential Revision: D34488397 |
Summary: Pull Request resolved: facebookexperimental#1639 Make `default` an optional property for `atom()` and `atomFamily()`. If not provided, the atom will initialize in a pending state until set. This can be convenient to avoid having to make an atom with some nullable type or trying to come up with a default placeholder or using something cryptic like `default: new Promise(() => {}),` Reviewed By: habond Differential Revision: D34488397 fbshipit-source-id: 4615b258146d2d3bed902008179868b179028ad3
|
This pull request was exported from Phabricator. Differential Revision: D34488397 |
drarmstr
force-pushed
the
export-D34488397
branch
from
35ff7a6 to
2d1a5c2
Compare
drarmstr
added a commit
to drarmstr/Recoil
that referenced
this pull request
Feb 26, 2022
Summary: Pull Request resolved: facebookexperimental#1639 Make `default` an optional property for atoms. If not provided the atom will initialize in a pending state until set. This can be convenient to avoid having to make an atom with some nullable type or trying to come up with a default placeholder or using something cryptic like `default: new Promise(() => {}),` Differential Revision: D34488397 fbshipit-source-id: e12e7f8cc05b342bf19307c2af8c06468e4283e1
drarmstr
added a commit
to drarmstr/Recoil
that referenced
this pull request
Feb 26, 2022
Summary: Pull Request resolved: facebookexperimental#1639 Make `default` an optional property for atoms. If not provided the atom will initialize in a pending state until set. This can be convenient to avoid having to make an atom with some nullable type or trying to come up with a default placeholder or using something cryptic like `default: new Promise(() => {}),` Differential Revision: D34488397 fbshipit-source-id: 0afdfa5a6b10bb407e791ebbb3f19da754530744
drarmstr
added a commit
to drarmstr/Recoil
that referenced
this pull request
Feb 26, 2022
Summary: Pull Request resolved: facebookexperimental#1639 Make `default` an optional property for atoms. If not provided the atom will initialize in a pending state until set. This can be convenient to avoid having to make an atom with some nullable type or trying to come up with a default placeholder or using something cryptic like `default: new Promise(() => {}),` Differential Revision: D34488397 fbshipit-source-id: b45ce52a634fbc24e696924757eb56aba333bb82
AlexGuz23
pushed a commit
to AlexGuz23/Recoil
that referenced
this pull request
Nov 3, 2022
Summary: Pull Request resolved: facebookexperimental/Recoil#1639 Make `default` an optional property for `atom()` and `atomFamily()`. If not provided, the atom will initialize in a pending state until set. This can be convenient to avoid having to make an atom with some nullable type or trying to come up with a default placeholder or using something cryptic like `default: new Promise(() => {}),` Reviewed By: habond Differential Revision: D34488397 fbshipit-source-id: 72b211b7b542ad51d2568cfeb103917b5d29fcea
snipershooter0701
pushed a commit
to snipershooter0701/Recoil
that referenced
this pull request
Mar 5, 2023
Summary: Pull Request resolved: facebookexperimental/Recoil#1639 Make `default` an optional property for `atom()` and `atomFamily()`. If not provided, the atom will initialize in a pending state until set. This can be convenient to avoid having to make an atom with some nullable type or trying to come up with a default placeholder or using something cryptic like `default: new Promise(() => {}),` Reviewed By: habond Differential Revision: D34488397 fbshipit-source-id: 72b211b7b542ad51d2568cfeb103917b5d29fcea
eagle2722
added a commit
to eagle2722/Recoil
that referenced
this pull request
Sep 21, 2024
Summary: Pull Request resolved: facebookexperimental/Recoil#1639 Make `default` an optional property for `atom()` and `atomFamily()`. If not provided, the atom will initialize in a pending state until set. This can be convenient to avoid having to make an atom with some nullable type or trying to come up with a default placeholder or using something cryptic like `default: new Promise(() => {}),` Reviewed By: habond Differential Revision: D34488397 fbshipit-source-id: 72b211b7b542ad51d2568cfeb103917b5d29fcea
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary:
Make
defaultan optional property for atoms. If not provided the atom will initialize in a pending state until set.This can be convenient to avoid having to make an atom with some nullable type or trying to come up with a default placeholder or using something cryptic like
default: new Promise(() => {}),Reviewed By: habond
Differential Revision: D34488397