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

feat: allow selenium server with internal webserver #621

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

daFish
Copy link

@daFish daFish commented Jan 18, 2024

Implements #590 by @dkarlovi.

This change allows the usage of an external selenium instance (e.g. via docker compose) to test against the internal webserver.

Todos:

  • Tests
  • Documentation

@dkarlovi
Copy link

permission of @dkarlovi

What "permission", thank you so much for working on this! \o/ :smile:

@daFish
Copy link
Author

daFish commented Jan 18, 2024

Coding Standards fails seem unrelated.

@dkarlovi Ok, let's rephrase it then. 😄

@dkarlovi
Copy link

dkarlovi commented Mar 1, 2024

@daFish this looks completed to me, why did you leave is as a draft?

@daFish
Copy link
Author

daFish commented Mar 4, 2024

@daFish this looks completed to me, why did you leave is as a draft?

I made a small info in the README about it. I will remove the draft status and this is good to go.

@daFish daFish marked this pull request as ready for review March 4, 2024 09:06
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.

None yet

2 participants