Skip to content

resolves #308 #309

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

Merged
merged 5 commits into from
Jul 16, 2024
Merged

resolves #308 #309

merged 5 commits into from
Jul 16, 2024

Conversation

waggledans
Copy link
Contributor

@waggledans waggledans commented Jul 15, 2024

One-line summary

Issue : #308

Description

  • Mapped US_EAST to us-east-4 datacenter that exists (even though it supports real mobile devices only)
  • Removed APAC
  • Updated us-west-1 ondemand endpoint base URL to https://ondemand.us-west-1.saucelabs.com from https://ondemand.saucelabs.com to be consistent

Types of Changes

What types of changes does your code introduce? Keep the ones that apply:

  • Configuration change

Review

List of tasks the reviewer must do to review the PR

  • Tests
  • Documentation
  • CHANGELOG

Deployment Notes

These should highlight any db migrations, feature toggles, etc.

@waggledans waggledans requested a review from titusfortner July 15, 2024 17:56
Copy link
Member

@diemol diemol left a comment

Choose a reason for hiding this comment

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

Thank you, @waggledans!

@diemol diemol merged commit bc8ba41 into main Jul 16, 2024
8 of 10 checks passed
@diemol diemol deleted the dans-dc-links branch July 16, 2024 07:23
@@ -7,7 +7,7 @@ sidebar_label: Data Center
While the Sauce Options class allows you to specify how you want your tests to run,
you may also want to adjust where your tests are run with the Sauce Session class.
By default, tests are executed on our US West Coast Data Center.
You can Specify US East (Headless only), Central EU, or APAC data centers with the Session class.
You can Specify US East (real mobile devices only), or Central EU data centers with the Session class.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's remove reference to US East since bindings don't support it, and we removed it from bindings

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