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

Dynamic path prefix #115

Merged
merged 14 commits into from
Aug 26, 2024
Merged

Dynamic path prefix #115

merged 14 commits into from
Aug 26, 2024

Conversation

bgentry
Copy link
Contributor

@bgentry bgentry commented Aug 25, 2024

Allows the UI assets to be built and have a dynamic path prefix injected at runtime. Also switches us to a new dev setup.

This was quite difficult to sort out, but fortunately https://github.com/amir20/dozzle/ had already blazed the trail here.

With these changes, we should be able to dramatically simplify the existing steps for #107, and solve it completely for all go get users if we're willing to publish pre-built release artifacts. Unlike with riverpro, these could be made publicly available and cached permanently by goproxy servers, so the hosting burden would be minimal.

Fixes #114.

@bgentry bgentry force-pushed the bg-dynamic-path branch 2 times, most recently from 51ad34a to 0e8a045 Compare August 26, 2024 00:58
@bgentry bgentry marked this pull request as ready for review August 26, 2024 01:48
@bgentry bgentry requested a review from brandur August 26, 2024 01:48
@bgentry bgentry merged commit 3251771 into master Aug 26, 2024
12 checks passed
@bgentry bgentry deleted the bg-dynamic-path branch August 26, 2024 16:09
@bgentry
Copy link
Contributor Author

bgentry commented Aug 26, 2024

Beautiful, just tested this with the latest Docker container from master (sha256:f9f5c9692cfe8c5c1edd88592e6885c4b116753ef139d0bbbc27badbbf027c10) and it's working perfectly both with and without PATH_PREFIX env 🙌

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.

PATH_PREFIX doesn't seem to work
2 participants