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

Make the port the client proxy proxies to configurable #346

Merged
merged 11 commits into from
May 9, 2024

Conversation

JulianKniephoff
Copy link
Member

This is based on #345, so that one needs to go in first.

Otherwise it does what it says on the tin.

This one is already covered by the one below it.
That is all of them.
Also add a missing proxy.
Also remove unused static data.
Note that this is not a thorough cleanup of the `test` folder.
@JulianKniephoff JulianKniephoff added the type:code-enhancement Internal improvements to the codebase label May 6, 2024
@JulianKniephoff JulianKniephoff force-pushed the configurable-proxy-port branch from 71ea051 to 52411fd Compare May 6, 2024 16:39
@JulianKniephoff JulianKniephoff changed the title Make the path the client proxy proxies to configurable Make the port the client proxy proxies to configurable May 7, 2024
JulianKniephoff added a commit to JulianKniephoff/opencast-admin-interface that referenced this pull request May 7, 2024
5000 is used by Control Center on macOS, so this will likely annoy
at least a few developers.

This is based on opencast#346, so that should be reviewed and merged first.

This comment was marked as resolved.

@JulianKniephoff JulianKniephoff force-pushed the configurable-proxy-port branch from 52411fd to b44b796 Compare May 8, 2024 07:21
JulianKniephoff added a commit to JulianKniephoff/opencast-admin-interface that referenced this pull request May 8, 2024
5000 is used by Control Center on macOS, so this will likely annoy
at least a few developers.

This is based on opencast#346, so that should be reviewed and merged first.
@ziegenberg
Copy link
Member

Works.

@ziegenberg ziegenberg merged commit 0aa6db5 into opencast:main May 9, 2024
1 check passed
@ziegenberg ziegenberg self-assigned this May 9, 2024
JulianKniephoff added a commit to JulianKniephoff/opencast-admin-interface that referenced this pull request May 14, 2024
5000 is used by Control Center on macOS, so this will likely annoy
at least a few developers.

This is based on opencast#346, so that should be reviewed and merged first.
JulianKniephoff added a commit to JulianKniephoff/opencast-admin-interface that referenced this pull request May 14, 2024
5000 is used by Control Center on macOS, so this will likely annoy
at least a few developers.

This is based on opencast#346, so that should be reviewed and merged first.
JulianKniephoff added a commit to JulianKniephoff/opencast-admin-interface that referenced this pull request May 15, 2024
5000 is used by Control Center on macOS, so this will likely annoy
at least a few developers.

This is based on opencast#346, so that should be reviewed and merged first.
JulianKniephoff added a commit to JulianKniephoff/opencast-admin-interface that referenced this pull request May 16, 2024
5000 is used by Control Center on macOS, so this will likely annoy
at least a few developers.

This is based on opencast#346, so that should be reviewed and merged first.
@JulianKniephoff JulianKniephoff deleted the configurable-proxy-port branch May 23, 2024 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:code-enhancement Internal improvements to the codebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants