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

WIP: Testing still... Revert "Get source, screenshot and windowSize concurrently" #1030

Closed

Conversation

dpgraham
Copy link
Contributor

@dpgraham dpgraham commented Jul 4, 2019

Reverts #916

Causing race conditions that cause the spinner to act indefinitely

@dpgraham dpgraham changed the title Revert "Get source, screenshot and windowSize concurrently" WIP: Testing still... Revert "Get source, screenshot and windowSize concurrently" Jul 4, 2019
@dpgraham
Copy link
Contributor Author

dpgraham commented Jul 8, 2019

This doesn't fix the issue.

@dpgraham dpgraham closed this Jul 8, 2019
@dpgraham dpgraham deleted the revert-916-dpgraham-concurrent-source-and-screenshot branch July 8, 2019 17:49
@vijayqeguy
Copy link

@dpgraham I am just providing more information to narrow down this issue. It works perfectly fine on v1.12.0 but not on v1.12.1. We just need to compare the changes done between these versions. I am trying from my side as well.

@danieltg
Copy link
Contributor

@dpgraham I've just tried it now and Perfecto cloud provider run properly after reverting commit #916 back.

@danieltg
Copy link
Contributor

danieltg commented Sep 3, 2019

@dpgraham Can you kindly retry the revert as per our testing it does fix the issue - at least for Perfecto cloud provider.

@dpgraham
Copy link
Contributor Author

dpgraham commented Sep 3, 2019

@danieltg I don't have time, but I'll just leave this as-is until I get around to it again

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants