Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

fix(Angular): add workaround for Safari / Webdriver error #16645

Merged
merged 1 commit into from
Jul 26, 2018

Conversation

Narretz
Copy link
Contributor

@Narretz Narretz commented Jul 25, 2018

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change?

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Fix/Feature: Docs have been added/updated
  • Fix/Feature: Tests have been added; existing tests pass

Other information:

@thorn0
Copy link
Contributor

thorn0 commented Jul 25, 2018

Why not wait until karma-runner/karma#3085 gets merged and released?

@Narretz
Copy link
Contributor Author

Narretz commented Jul 25, 2018

Oh cool I didn't know about this PR. I thought this must be fixed inside Webdriver or by Saucelabs

@Narretz Narretz merged commit 6b915ad into angular:master Jul 26, 2018
@Narretz
Copy link
Contributor Author

Narretz commented Jul 26, 2018

We will remove this workaround once this is fixed upstream

@Narretz Narretz deleted the fix-safari-webdriver branch July 26, 2018 08:43
@thorn0
Copy link
Contributor

thorn0 commented Oct 22, 2018

Karma 3.1.0 has been released with a fix for this.

Narretz added a commit to Narretz/angular.js that referenced this pull request Oct 25, 2018
This allows us to remove the workaround added in angular#16645
Narretz added a commit to Narretz/angular.js that referenced this pull request Oct 25, 2018
This allows us to remove the workaround added in angular#16645
Narretz added a commit to Narretz/angular.js that referenced this pull request Nov 2, 2018
This allows us to remove the workaround added in angular#16645
Narretz added a commit that referenced this pull request Nov 23, 2018
This allows us to remove the workaround added in #16645
Narretz added a commit that referenced this pull request Nov 26, 2018
This allows us to remove the workaround added in #16645
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants