Skip to content

Conversation

@mcdurdin
Copy link
Member

@mcdurdin mcdurdin commented Feb 13, 2025

  • Splits the server-side and client-side references to keyman sites so that docker-based PHP can reference host.docker.internal on development machines for cross-references.
  • TODO: update to BOOTSTRAP_VERSION=v0.17 in build.sh before merge.

Relates-to: keymanapp/shared-sites#53
Relates-to: keymanapp/keyman.com#545
Relates-to: keymanapp/api.keyman.com#272

* Splits the server-side and client-side references to keyman sites so
  that docker-based PHP can reference host.docker.internal on
  development machines for cross-references.

* TODO: update to `BOOTSTRAP_VERSION=v0.17` in build.sh before merge.

Relates-to: keymanapp/shared-sites#53
Relates-to: keymanapp/keyman.com#545
Relates-to: keymanapp/api.keyman.com#272
@mcdurdin mcdurdin added this to the B18S1 milestone Feb 13, 2025
mcdurdin added a commit to keymanapp/help.keyman.com that referenced this pull request Feb 13, 2025
* Splits the server-side and client-side references to keyman sites so
  that docker-based PHP can reference host.docker.internal on
  development machines for cross-references.

* TODO: update to `BOOTSTRAP_VERSION=v0.17` in build.sh before merge.

Relates-to: keymanapp/shared-sites#53
Relates-to: keymanapp/keyman.com#545
Relates-to: keymanapp/api.keyman.com#272
Relates-to: keymanapp/keymanweb.com#129
@mcdurdin mcdurdin marked this pull request as ready for review February 13, 2025 07:34
@mcdurdin mcdurdin requested a review from darcywong00 February 13, 2025 07:34
@darcywong00 darcywong00 modified the milestones: B18S1, B18S2 Feb 15, 2025
Copy link
Contributor

@darcywong00 darcywong00 left a comment

Choose a reason for hiding this comment

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

lgtm

@mcdurdin mcdurdin merged commit a32e060 into master Feb 17, 2025
2 checks passed
@mcdurdin mcdurdin deleted the refactor/keyman-hosts-server-side-links branch February 17, 2025 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants