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

Enable editing process models in GitHub Codespaces #2

Closed
wants to merge 6 commits into from
Closed

Enable editing process models in GitHub Codespaces #2

wants to merge 6 commits into from

Conversation

mogul
Copy link

@mogul mogul commented Oct 3, 2024

[oops]

mogul added 6 commits October 3, 2024 13:13
GitHub Codespaces puts each forwarded port on a separate subdomain. That
won't work, per the SpiffWorkflow documentation:
https://spiff-arena.readthedocs.io/en/latest/DevOps_installation_integration/path_based_routing.html

So we introduce an nginx proxy container to implement path-based
routing. Unfortunately path-based routing doesn't work with "docker
compose up" locally because code in the frontend explicitly assumes
that the backend is on a neighboring port if the hostname is
"localhost". In spite of that, I'm pushing this change up to see if it
fixes the behavior in a GitHub Codespace.
@mogul mogul closed this Oct 3, 2024
@mogul
Copy link
Author

mogul commented Oct 3, 2024

Oops, wrong base repo selected.

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.

1 participant