-
Notifications
You must be signed in to change notification settings - Fork 77
Fix various errors #37
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
base: master
Are you sure you want to change the base?
Conversation
@sylvainDNS is attempting to deploy a commit to the void Team on Vercel. A member of the Team first needs to authorize it. |
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/void-team/ui-playbook/7JUHk9q7qhxWJztV9pg1RbCLZ6F5 |
const [added, setAdded] = React.useState(false); | ||
const [count, setCount] = React.useState(0); | ||
|
||
React.useEffect(() => { | ||
const id = setInterval(() => { | ||
setAdded((prevAdded) => { | ||
if (prevAdded === false) { | ||
setTimeout(() => { | ||
setCount((prevCount) => prevCount + 1); | ||
}, 1000); | ||
} | ||
return !prevAdded; | ||
}); | ||
}, 1000); | ||
React.useEffect(() => { | ||
const id = setInterval(() => { | ||
setAdded((prevAdded) => { | ||
if (prevAdded === false) { | ||
setTimeout(() => { | ||
setCount((prevCount) => prevCount + 1); | ||
}, 1000); | ||
} | ||
return !prevAdded; | ||
}); | ||
}, 1000); | ||
|
||
return () => { | ||
clearInterval(id); | ||
}; | ||
}, []); | ||
return () => { | ||
clearInterval(id); | ||
}; | ||
}, []); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you revert the cosmical changes? Wondering why these got changed 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the slow response.
The problem comes from prettier configured like this :
// .prettierrc.js
useTabs: true
tabWidth: 4
I can fix it, what config do you want ?
This PR includes several fixes :
<MdxLink />
component to fix the deprecated use of Next<Link />
with a string child. If someone has a better workaround, feel free to tell me! 😉