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

Update live-cross-browser-testing.md #3053

Merged
merged 2 commits into from
Jan 10, 2025
Merged

Conversation

lee-snyder133
Copy link
Contributor

Updated section: ### Time Limits and Timeouts for Cross Browser Testing

Had spelling error in "Browser"

Description

Under the section ### Time Limits and Timeouts for Cross Browser Testing

  • Updated Broswer to Browser

Updated grammar in section:

  • Cross Browser tests for free users have a 11 minute limit from session start.
  • Cross Browser tests for free users have an 11-minute limit from the session start.

Updated grammar in section:

  • Cross Browser tests for all accounts will timeout after 90 seconds if the session is not active (that is, has no active channels).
  • Cross Browser tests for all accounts will timeout after 90 seconds if the session is not active (that is, it has no active channels)

Motivation and Context

To eliminate grammar errors in our documentation

Types of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation fix (typos, incorrect content, missing content, etc.)

Updated section: ### Time Limits and Timeouts for Cross Browser Testing

Had spelling error in "Browser"
@lee-snyder133 lee-snyder133 requested a review from a team as a code owner January 6, 2025 22:52
Copy link

github-actions bot commented Jan 6, 2025

Deploy preview ready for 3053!
https://docs.dev.saucelabs.net/pr-preview/pr-3053

Copy link
Collaborator

@si-net si-net left a comment

Choose a reason for hiding this comment

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

Approved! thank you @lee-snyder133 feel free to merge.

Copy link

Deploy preview ready for 3053!
https://docs.dev.saucelabs.net/pr-preview/pr-3053

@lee-snyder133 lee-snyder133 merged commit efc97a1 into main Jan 10, 2025
3 checks passed
@lee-snyder133 lee-snyder133 deleted the lee-snyder133-patch-2 branch January 10, 2025 15:29
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