-
Notifications
You must be signed in to change notification settings - Fork 73
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 instructions and best practices to the README for dependent props #342
Conversation
Looks good to me, thank you! |
What I used was two export default withOnyx({
report: {
key: ({reportID) => `${ONYXKEYS.COLLECTION.REPORT}${reportID}`,
}})
withOnyx({
policy: {
key: ({report}) => `${ONYXKEYS.COLLECTION.POLICY}${report.policyID}`,
},
})(App); Is this different from single |
Co-authored-by: Ana Margarida Silva <ams-amargarida@hotmail.com>
51956f7
@parasharrajat It functionally results in the same end result, but with two |
Glad to know. Thanks for the explanation. |
@mountiny @hayata-suenaga do one of you want to go ahead and merge this? |
Coming from this slack convo: https://expensify.slack.com/archives/C01GTK53T8Q/p1694710295343199
cc @mountiny @marcaaron @allroundexperts