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

Updating Test Capabilities to be in Correct Section and Updated Typos #3047

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

Conversation

lee-snyder133
Copy link

Description

Change 1 | Updated sentence to remove line break:

BEFORE
See the list of available time zone identifiers f
or
more details. The same behavior could be achieved by providing a custom

AFTER
See the list of available time zone identifiers for more details. The same behavior could be achieved by providing a custom

Change 2 | Updated code block in appium real device doc to remove space:
BEFORE
"appium: noReset" : "true",

AFTER
"appium:noReset" : "true",

Change 3 | Moved two capabilities as they are not supported on RDC:

  • recordVideo
  • videoUploadOnPass

Moved FROM: Desktop and Mobile Capabilities: Sauce-Specific – Optional
TO: Desktop and Virtual Device Capabilities: Sauce-Specific – Optional

Motivation and Context

To improve readability of docs and to ensure features are properly categorized so users know how to use them

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.)

Removed the space that was in this code block for appium:noReset

```js
"appium:noReset" : "true",
"sauce:options" : {
  "cacheId" : "jnc0x1256",
}
```
Moved the following 2 capabilities:
recordVideo
videoUploadOnPass
FROM section: Desktop and Mobile Capabilities: Sauce-Specific – Optional
TO section: Desktop and Virtual Device Capabilities: Sauce-Specific – Optional
As they are not supported on Real Devices.
Under appium:appTimeZone, line 883, there was a text formatting issue:
f
or more details.

Updated to:
for more details.
@lee-snyder133 lee-snyder133 requested a review from a team as a code owner December 13, 2024 16:21
Copy link

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

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.

2 participants