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

Cross-reference visiting external sites anti-pattern #6044

Merged

Conversation

MikeMcC399
Copy link
Contributor

Issue

  • The Anti-Pattern description Visiting External Sites and the section in Step 1: Visit a page with the title "Testing Apps You Don't Control" are related and not linked.
  • There is no bookmark to reference "Testing Apps You Don't Control".

Change

  1. Move "Testing Apps You Don't Control" down the page on Your First Test to allow it to be referenced.
  2. Cross-reference the two sections that refer to visiting external sites, between the two pages:

Copy link

netlify bot commented Dec 15, 2024

👷 Deploy request for cypress-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit cabc0ad

@cypress-app-bot
Copy link
Collaborator

@MikeMcC399 MikeMcC399 marked this pull request as ready for review December 15, 2024 14:55
@jennifer-shehane jennifer-shehane merged commit 40f4df4 into cypress-io:main Dec 17, 2024
5 checks passed
@MikeMcC399 MikeMcC399 deleted the x-ref/anti-pattern-external-site branch December 18, 2024 08:13
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