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

chore: correct broken links in docs #15359

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

tez3998
Copy link

@tez3998 tez3998 commented Oct 24, 2024

Summary

I corrected some broken links in docs.

The pages I edited are as follows:

  • More Resources
    • React Native
  • Testing Web Frameworks
    • Running AngularJS Tests with Jest @ AngularJS
    • How to test Express.js with Jest and Supertest @ Express.js (reverted & corrected the link)
    • Unit Testing @ GatsbyJS

Test plan

As shown in the attached video, I tested locally:

jest_link_fix.mp4

The example for Express.js was removed in the following pull request due to a broken link:
jestjs#15270

Although the domain is different,
I found what appears to be the same blog based on the author’s name and title.
Therefore, I reverted the Express.js section and corrected the link.
Copy link

linux-foundation-easycla bot commented Oct 24, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

Copy link

netlify bot commented Oct 24, 2024

Deploy Preview for jestjs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 7220ddb
🔍 Latest deploy log https://app.netlify.com/sites/jestjs/deploys/671a7c62834bbf000863f351
😎 Deploy Preview https://deploy-preview-15359--jestjs.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 configuration.

@tez3998
Copy link
Author

tez3998 commented Oct 25, 2024

If this PR is to be merged, #15311 must be closed without merge, because it removes the link itself.

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.

1 participant