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

Add <script> tags in the download page for easy embed #44

Merged
merged 1 commit into from
Jun 3, 2013
Merged

Add <script> tags in the download page for easy embed #44

merged 1 commit into from
Jun 3, 2013

Conversation

vjeux
Copy link
Contributor

@vjeux vjeux commented Jun 3, 2013

image

zpao added a commit that referenced this pull request Jun 3, 2013
Add <script> tags in the download page for easy embed
@zpao zpao merged commit dfdf1f8 into facebook:master Jun 3, 2013
@vjeux vjeux deleted the script_download branch June 3, 2013 00:29
taneliang referenced this pull request in MLH-Fellowship/react Aug 17, 2020
Summary
---

Resolves #44.

Intended to highlight all related React events. However, it looks like
separate promises can have the same wakeable ID, which is a bit strange.

TODO
---

* Ensure that we are correctly understanding wakeable IDs, i.e. that
  a unique resource/promise is a unique wakeable.
* Check if `unstable_createResource` caches promises.

Test Plan
---

* `yarn lint`
* `yarn flow`: no errors in changed code
taneliang referenced this pull request in MLH-Fellowship/react Aug 19, 2020
Summary
---

Resolves #44.

Intended to highlight all related React events. However, it looks like
separate promises can have the same wakeable ID, which is a bit strange.

TODO
---

* Ensure that we are correctly understanding wakeable IDs, i.e. that
  a unique resource/promise is a unique wakeable.
* Check if `unstable_createResource` caches promises.

Test Plan
---

* `yarn lint`
* `yarn flow`: no errors in changed code
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