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

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

Merged
Narretz merged 1 commit into
angular:masterfrom
Narretz:fix-safari-webdriver
Jul 26, 2018
Merged

Narretz merged 1 commit into
angular:masterfrom
Narretz:fix-safari-webdriver

Conversation

@Narretz

@Narretz Narretz commented Jul 25, 2018

Copy link
Copy Markdown
Contributor

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

thorn0 commented Jul 25, 2018

Copy link
Copy Markdown
Contributor

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

@Narretz

Narretz commented Jul 25, 2018

Copy link
Copy Markdown
Contributor Author

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

Narretz commented Jul 26, 2018

Copy link
Copy Markdown
Contributor Author

We will remove this workaround once this is fixed upstream

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

thorn0 commented Oct 22, 2018

Copy link
Copy Markdown
Contributor

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