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

Invalid/Expired/Modified URL #699

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Invalid/Expired/Modified URL #699

wants to merge 1 commit into from

Conversation

HeyProtagonist
Copy link

line: 40 >> [whatthefork.is/closure](https://whatthefork.is/closure). >> [whatthefuck.is/closure](https://whatthefuck.is/closure).
Copy link
Contributor

@Aprillion Aprillion left a comment

Choose a reason for hiding this comment

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

🙄🤦‍♂️

@HeyProtagonist
Copy link
Author

🙄🤦‍♂️

I know it's silly! But took me some time find this damn good article!

@Aprillion
Copy link
Contributor

Aprillion commented Jul 25, 2021

oh, wait, this is a serious PR and not just trolling 🤔 looks like someone pulled the plug from whatthefork.is and now that is a dead link 😱

I have a feeling that Kent might prefer https://developer.mozilla.org/en-US/docs/Web/JavaScript/Closures over the new domain whatthefuck.is

@kentcdodds
Copy link
Owner

Yup. I was hoping that @daydream05 would get around to renewing the domain (https://github.com/kentcdodds/advanced-react-hooks/issues/141) so we could keep it as-is. I just have a problem with the f-word.

If the fork version isn't updated soon then yeah, I think we'll probably just go with the mdn article.

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.

3 participants