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

report subscribing components #714

Closed

Conversation

drarmstr
Copy link
Contributor

@drarmstr drarmstr commented Nov 5, 2020

Summary:
Report components which are subscribing to atoms and selectors via useGetRecoilValueInfo().

Note that Snapshot#getInfo() will not report any subscribing components at this time.

This currently reports a string of the component name. But, we likely want to change that to something more specific/useful.

Reviewed By: maxijb

Differential Revision: D24718812

Summary: Add `useRecoilValueInfo()` hook which acts similar to `getInfo()` method on `Snapshot`, except peeks the info for the current state.

Differential Revision: D24718803

fbshipit-source-id: 3712ea739db99f6347e5459782fb196be560265c
Summary:
Report components which are subscribing to atoms and selectors via `useGetRecoilValueInfo()`.

Note that `Snapshot#getInfo()` will not report any subscribing components at this time.

This currently reports a string of the component name.  But, we likely want to change that to something more specific/useful.

Reviewed By: maxijb

Differential Revision: D24718812

fbshipit-source-id: 84489bcff5a2df7a999648a5661f55657b0bd083
@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 Nov 5, 2020
@facebook-github-bot
Copy link
Contributor

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

@drarmstr drarmstr self-assigned this Nov 5, 2020
@drarmstr drarmstr added devtools enhancement New feature or request labels Nov 5, 2020
@drarmstr drarmstr deleted the export-D24718812 branch November 17, 2020 03:47
@drarmstr
Copy link
Contributor Author

Document in #744

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in c6d786b.

AlexGuz23 pushed a commit to AlexGuz23/Recoil that referenced this pull request Nov 3, 2022
Summary:
Pull Request resolved: facebookexperimental/Recoil#714

Report components which are subscribing to atoms and selectors via `useGetRecoilValueInfo()`.

Note that `Snapshot#getInfo()` will not report any subscribing components at this time.

This currently reports a string of the component name.  But, we likely want to change that to something more specific/useful.

Reviewed By: maxijb

Differential Revision: D24718812

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

Report components which are subscribing to atoms and selectors via `useGetRecoilValueInfo()`.

Note that `Snapshot#getInfo()` will not report any subscribing components at this time.

This currently reports a string of the component name.  But, we likely want to change that to something more specific/useful.

Reviewed By: maxijb

Differential Revision: D24718812

fbshipit-source-id: 8f8bfb298592e2d2d78b845287269805d8ad18f9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. devtools enhancement New feature or request fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants