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

fix: typo in useFocusEffect with async function error #10688

Conversation

lipidity
Copy link
Contributor

Unmatched parentheses in the sample code included in the error message when an async function is passed to useFocusEffect.

Motivation

People who copy/paste from the error message and adapt it to their requirements would have a syntax error.

Test plan

Pass an async function to useFocusEffect, check that the parens are balanced in the sample code shown in the error message.

Unmatched parentheses in the sample code when async function passed to useFocusEffect
@github-actions
Copy link

Hey lipidity! Thanks for opening your first pull request in this repo. If you haven't already, make sure to read our contribution guidelines.

@netlify
Copy link

netlify bot commented Jul 11, 2022

Deploy Preview for react-navigation-example ready!

Name Link
🔨 Latest commit aa043af
🔍 Latest deploy log https://app.netlify.com/sites/react-navigation-example/deploys/62cba29045719a0008b4abad
😎 Deploy Preview https://deploy-preview-10688--react-navigation-example.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Member

@kacperkapusciak kacperkapusciak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@kacperkapusciak kacperkapusciak merged commit 9203045 into react-navigation:main Jul 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants