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

Implement uniform URL for connecting remote browsers (closes #1476) #1478

Merged
merged 1 commit into from
May 26, 2017

Conversation

AndreyBelym
Copy link
Contributor

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit 9f8d92c have failed. See details:

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit 1723c83 have failed. See details:

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit ffbc6eb have failed. See details:

1 similar comment
@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit ffbc6eb have failed. See details:

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit ffbc6eb have passed. See details:

Copy link
Contributor

@AlexanderMoskovkin AlexanderMoskovkin left a comment

Choose a reason for hiding this comment

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

LGTM but it necessary to check the string with a techwriter


if (remoteCount) {
log.hideSpinner();

var description = dedent(`
Connecting ${remoteCount} remote browser(s)...
Navigate to the appropriate URL from each of the remote browsers.
Navigate to the following URL from each of the remote browsers.
Copy link
Contributor

Choose a reason for hiding this comment

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

Navigate to the following URL from each remote browser.

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit 87ad77a have passed. See details:

@AndreyBelym
Copy link
Contributor Author

FPR

@AlexanderMoskovkin AlexanderMoskovkin merged commit 60ebe59 into DevExpress:master May 26, 2017
kirovboris pushed a commit to kirovboris/testcafe-phoenix that referenced this pull request Dec 18, 2019
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.

4 participants